Theme doesn't work on the map with commercial layers

[liuar]

Steps:

1. Open one flexible web layout with commercial layers.
2. Theme -> Set theme rule -> Apply

Results: Nothing changed in web layout.

Expected results: Map should display as Theme rule.

[liuar]

Apply theme invoke reloadMap in Generic.js which will reload all the commercial layers in current map. In loadMap method in Generic.js at line 77, if a commericial layer already exsits, it will be destroy then create a new one . But after the layer destroyed, the point to the layer is not set to null which will ignore the layer re-creation and cause a exception like "null reference exception" when register events to this.oLayerOL.events.

[liuar]

Fix typo of my first commet section.

Apply theme invokes reloadMap in Generic.js which will reload all the commercial layers in current map. In loadMap method in Generic.js at line 77, if a commercial layer already exists, it will be destroied then create a new one . But after the layer destroyed, the pointer to the layer is not set to null which will ignore the layer’s re-creation and will cause an exception like "null reference exception" when register events to this.oLayerOL.events.

[zjames]

ok, please commit.

[tomfukushima]

Chris can you commit this?

[chrisclaydon]

Submitted patch and converted the tabs it contains to spaces. In the future, please try to ensure that all tabs are converted to spaces before creating patches. Thx.

http://trac.osgeo.org/fusion/changeset/1983