[jira] Updated: (WICKET-2332) Open up Markup ctor and MarkupContainer#renderNext

2009-06-18 Thread Erik van Oosten (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erik van Oosten updated WICKET-2332:


Attachment: WICKET-2332-2.patch

Second patch also makes  org.apache.wicket.markup.Markup.makeImmutable() 
public. Again, please apply in trunk.

> Open up Markup ctor and MarkupContainer#renderNext
> --
>
> Key: WICKET-2332
> URL: https://issues.apache.org/jira/browse/WICKET-2332
> Project: Wicket
>  Issue Type: Improvement
>  Components: wicket
>Affects Versions: 1.4-RC4
>Reporter: Erik van Oosten
> Fix For: 1.4-RC6
>
> Attachments: WICKET-2332-2.patch, WICKET-2332.patch
>
>
> -1- Change org.apache.wicket.markup.Markup#Markup(): from default to public
> -2- Change org.apache.wicket.MarkupContainer#renderNext(MarkupStream): from 
> private to protected (it stays final)
> Rationale from the dev e-mail list:
> Erik van Oosten wrote:
> ---
> Hi,
> I am writing a Swing like layout manager (with MIG layout manager's API 
> as inspiration). The layout manager will generate HTML, but also has the 
> ability to override this with HTML defined by the user. I have a 
> prototype that does this. It takes elements from the current 
> Markupstream, adds the elements that are missing, and then some more to 
> facilitate the layouting (well the last step is next on my todo list). 
> These elements are then put in a new markupstream that is used to drive 
> onComponentTagBody of the layout component.
> For this to work I had to open up 2 things:
> - org.apache.wicket.markup.Markup#Markup(): from default to public
>   I could have written my own implementation of IMarkup, but that seems 
> a bit overkill as I need exactly what is in Markup.
> - org.apache.wicket.MarkupContainer#renderNext(MarkupStream): from 
> private to protected (don't care if it stays final)
>   My onComponentTagBody implementation is loosely based on method 
> org.apache.wicket.MarkupContainer#renderAssociatedMarkup(String,String). 
> One of the methods I need to call to render children seems to be renderNext.
> Could these two be opened up?
> ---
> Juergen Donnerstag wrote:
> ---
> I don't mind open up both, as long as renderNext() stays final and we
> tag it as "THIS IS WICKET INTERNAL".
> ---

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (WICKET-2332) Open up Markup ctor and MarkupContainer#renderNext

2009-06-18 Thread Erik van Oosten (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erik van Oosten updated WICKET-2332:


Fix Version/s: (was: 1.4-RC5)
   1.4-RC6

> Open up Markup ctor and MarkupContainer#renderNext
> --
>
> Key: WICKET-2332
> URL: https://issues.apache.org/jira/browse/WICKET-2332
> Project: Wicket
>  Issue Type: Improvement
>  Components: wicket
>Affects Versions: 1.4-RC4
>Reporter: Erik van Oosten
> Fix For: 1.4-RC6
>
> Attachments: WICKET-2332.patch
>
>
> -1- Change org.apache.wicket.markup.Markup#Markup(): from default to public
> -2- Change org.apache.wicket.MarkupContainer#renderNext(MarkupStream): from 
> private to protected (it stays final)
> Rationale from the dev e-mail list:
> Erik van Oosten wrote:
> ---
> Hi,
> I am writing a Swing like layout manager (with MIG layout manager's API 
> as inspiration). The layout manager will generate HTML, but also has the 
> ability to override this with HTML defined by the user. I have a 
> prototype that does this. It takes elements from the current 
> Markupstream, adds the elements that are missing, and then some more to 
> facilitate the layouting (well the last step is next on my todo list). 
> These elements are then put in a new markupstream that is used to drive 
> onComponentTagBody of the layout component.
> For this to work I had to open up 2 things:
> - org.apache.wicket.markup.Markup#Markup(): from default to public
>   I could have written my own implementation of IMarkup, but that seems 
> a bit overkill as I need exactly what is in Markup.
> - org.apache.wicket.MarkupContainer#renderNext(MarkupStream): from 
> private to protected (don't care if it stays final)
>   My onComponentTagBody implementation is loosely based on method 
> org.apache.wicket.MarkupContainer#renderAssociatedMarkup(String,String). 
> One of the methods I need to call to render children seems to be renderNext.
> Could these two be opened up?
> ---
> Juergen Donnerstag wrote:
> ---
> I don't mind open up both, as long as renderNext() stays final and we
> tag it as "THIS IS WICKET INTERNAL".
> ---

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (WICKET-2332) Open up Markup ctor and MarkupContainer#renderNext

2009-06-18 Thread Erik van Oosten (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erik van Oosten updated WICKET-2332:


Attachment: WICKET-2332.patch

Please apply path in trunk.

> Open up Markup ctor and MarkupContainer#renderNext
> --
>
> Key: WICKET-2332
> URL: https://issues.apache.org/jira/browse/WICKET-2332
> Project: Wicket
>  Issue Type: Improvement
>  Components: wicket
>Affects Versions: 1.4-RC4
>Reporter: Erik van Oosten
> Fix For: 1.4-RC5
>
> Attachments: WICKET-2332.patch
>
>
> -1- Change org.apache.wicket.markup.Markup#Markup(): from default to public
> -2- Change org.apache.wicket.MarkupContainer#renderNext(MarkupStream): from 
> private to protected (it stays final)
> Rationale from the dev e-mail list:
> Erik van Oosten wrote:
> ---
> Hi,
> I am writing a Swing like layout manager (with MIG layout manager's API 
> as inspiration). The layout manager will generate HTML, but also has the 
> ability to override this with HTML defined by the user. I have a 
> prototype that does this. It takes elements from the current 
> Markupstream, adds the elements that are missing, and then some more to 
> facilitate the layouting (well the last step is next on my todo list). 
> These elements are then put in a new markupstream that is used to drive 
> onComponentTagBody of the layout component.
> For this to work I had to open up 2 things:
> - org.apache.wicket.markup.Markup#Markup(): from default to public
>   I could have written my own implementation of IMarkup, but that seems 
> a bit overkill as I need exactly what is in Markup.
> - org.apache.wicket.MarkupContainer#renderNext(MarkupStream): from 
> private to protected (don't care if it stays final)
>   My onComponentTagBody implementation is loosely based on method 
> org.apache.wicket.MarkupContainer#renderAssociatedMarkup(String,String). 
> One of the methods I need to call to render children seems to be renderNext.
> Could these two be opened up?
> ---
> Juergen Donnerstag wrote:
> ---
> I don't mind open up both, as long as renderNext() stays final and we
> tag it as "THIS IS WICKET INTERNAL".
> ---

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.