Summary: | [Chameleon - Widget] Popup assumes invalid attribute in SharedResource | ||
---|---|---|---|
Product: | Chameleon | Reporter: | Paul Spencer <pspencer@dmsolutions.ca> |
Component: | Widget | Assignee: | Julien-Samuel Lacroix <jlacroix@mapgears.com> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | bartvde@osgis.nl, chameleon-dev@lists.maptools.org, kueda@wso.williams.edu |
Priority: | P1 | ||
Version: | 2.2 | ||
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: |
I think the right way to fix this bug is to change the structure of the popup shared resource so that there is a subtag, as in: <cwc2 type="SharedResource" name="PopupCSS"> <url value="<url to css file"/> </cwc2> We will need to make sure that the code checks both the new case and the old case (value="" in the cwc2 tag) in 2.3/2.4 and then we could eventually remove it in the following version. In the current version, we need to emit an error if the old value="" in the cwc2 tag is used. In the next version, we can remove those checks. Ideally, we would also change the Popup.php so that you could specify the name of the shared resource to use, popupcssresource="MyPopupCSS", but that is a less critical change at this point.
*** Bug 1110 has been marked as a duplicate of this bug. ***
Popup now use the URL subtag in the Shared Resource PopupCSS. <cwc2 type="SharedResource" name="PopupCSS"> <url value="<url to css file>"/> </cwc2> It still support the VALUE in the shared resource tag. However, an error is returned. Marking as FIXED