Opened 14 years ago

Closed 14 years ago

#359 closed defect (fixed)

Theme on raster layer cause an error

Reported by: liuar Owned by: madair
Priority: P2 Milestone: Future
Component: Widgets Version: 1.1.1
Severity: Major Keywords:
Cc: jennyhe, jie.tan@… Browser: All
External ID: 1302595 Operating System: All
state: New

Description

  1. Create a flexible weblayout with raster layer
  2. View it in browser
  3. Click Theme widget
  4. An error occurs as the attached image

Attachments (2)

Raster.JPG​ (66.0 KB ) - added by liuar 14 years ago.
thememain.patch​ (534 bytes ) - added by liuar 14 years ago.

Download all attachments as: .zip

Change History (4)

by liuar, 14 years ago

Attachment: Raster.JPG​ added

comment:1 by liuar, 14 years ago

[Analysis]
For layers which don't contain the geometry property, the layer's name will not be record in layerNames in theme.php
So in thememain.php the layerSelect.value will be null, and the layerInfo request will fail.

[Resolution/Workaround]
To resolve this problem, I checked the layerSelect.value, if it's null, just return.

by liuar, 14 years ago

Attachment: thememain.patch​ added

comment:2 by liuar, 14 years ago

Cc: jennyhe jie.tan@… added
External ID: → 1302595
Resolution: → fixed
Status: new → closed

Thank Mike for the review.
Fixed at http://trac.osgeo.org/fusion/changeset/2064

Note: See TracTickets for help on using tickets.