[#SHALE-275] DialogNavigtionHandler breaks the implicit Decorator pattern

Shale

DialogNavigtionHandler breaks the implicit Decorator pattern

Created: 04/Sep/06 11:45 PM Updated: 23/Jan/07 04:40 PM

Return to search

Component/s:

Dialog, Sandbox

Affects Version/s:

1.0.3

Fix Version/s:

1.0.4

Issue Links:

Reference

This issue is related to:

~~SHALE-10~~ [shale] IFrame does not work properly inside Shale dialog

Description

« Hide

Because DialogNavigationHandler makes use of the Decorator pattern, it is strongly implied that navigation outcomes that are not handled by it will be delegated to the wrapped instance of NavigationHandler. This is not happening in the current nightly code for dialog (nor is it happening in the sandbox rewrite.)

All

Comments

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

sean schofield added a comment - 04/Sep/06 11:46 PM

This issue relates somewhat to ~~SHALE-10~~. Once addressed we should revisit that issue and see if there are any remaining problems.

[ Permlink | « Hide ]

sean schofield added a comment - 04/Sep/06 11:47 PM

I have created a test case that verifies the problem and solution. Test case introduces a shale-test and jmock dependency but this is just for testing so I think we're fine with this result.