Bugs item #1561701, was opened at 2006-09-19 15:47
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1561701&group_id=119783

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: core
Group: 1.2
Status: Open
Resolution: None
Priority: 5
Submitted By: Alex Burgel (aburgel)
Assigned to: Nobody/Anonymous (nobody)
Summary: BodyTagAttributeModifier does not work with Page

Initial Comment:
in wicket 1.2.2, adding a BodyTagAttributeModifier to a
Page will have no effect.

the reason is that BodyTagAttributeModifier.newValue()
tries to find the parent Page object of the
behaviorOwner, which in this case is a Page object.
unfortunately, the implementation of findParent() does
not work in this case.

instead, the method Component.getPage() should be used,
as this correctly handles the case when the component
is itself a Page.


a separate minor issue, the javadoc for BodyContainer
says that null is an allowed value for behaviorOwner
for methods addOnLoadModifier, addOnUnloadModifier.

you actually need a valid behaviorOwner otherwise
calling these methods will have no effect, as the
BodyTagAttributeModifier that gets created will be ignored.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1561701&group_id=119783

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Wicket-develop mailing list
Wicket-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to