Opened 6 years ago

Closed 6 years ago

#415 closed defect (fixed)

LayerLegend update can cause js error

Reported by: bartvde Owned by:
Priority: major Milestone: 1.1
Component: GeoExt.LayerLegend Version: 1.0
Keywords: Cc:
State: Commit

Description

This issue was initially reported here:

http://getsatisfaction.com/opengeo/topics/some_wms_server_give_error_in_layerlegend_update

It seems garbage collection is causing this, since items.get(0) is actually the image and not the label in my debugging sessions and that's why it fails.

Attachments (1)

geoext-415.patch (637 bytes) - added by bartvde 6 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 6 years ago by bartvde

  • Type changed from enhancement to defect

Changed 6 years ago by bartvde

comment:2 Changed 6 years ago by bartvde

  • State changed from None to Review

Tests still pass in Safari 5, thanks for any review.

comment:3 Changed 6 years ago by ahocevar

  • State changed from Review to Commit

Clean patch. Please commit.

comment:4 Changed 6 years ago by bartvde

  • Resolution set to fixed
  • Status changed from new to closed

(In [2643]) LayerLegend update can cause js error, r=ahocevar (closes #415)

Note: See TracTickets for help on using tickets.