[jira] Created: (WICKET-2000) AjaxRequestTarget escapes ] to ]^

2009-01-02 Thread Jan Kriesten (JIRA)
AjaxRequestTarget escapes ] to ]^
-

 Key: WICKET-2000
 URL: https://issues.apache.org/jira/browse/WICKET-2000
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.4-RC1
Reporter: Jan Kriesten
 Fix For: 1.4-RC2


Current 1.4 trunk 'messes' with escaping ']' to ']^' on 
AjaxRequestTarget.appendJavascript which leeds to non-functional Javascript 
when using array indices on variables.


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



[jira] Created: (WICKET-1997) TextFilteredPropertyColumn needs different generic for FilterModel

2008-12-29 Thread Jan Kriesten (JIRA)
TextFilteredPropertyColumn needs different generic for FilterModel
--

 Key: WICKET-1997
 URL: https://issues.apache.org/jira/browse/WICKET-1997
 Project: Wicket
  Issue Type: Bug
  Components: wicket-extensions
Affects Versions: 1.4-RC1
Reporter: Jan Kriesten


Like ChoiceFilteredPropertyColumn: TextFilteredPropertyColumn needs separate 
generic for the FilterModel 

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



[jira] Created: (WICKET-1965) Remove final from MarkupCache#clear()

2008-12-01 Thread Jan Kriesten (JIRA)
Remove final from MarkupCache#clear()
-

 Key: WICKET-1965
 URL: https://issues.apache.org/jira/browse/WICKET-1965
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.4-RC1
Reporter: Jan Kriesten
Priority: Minor
 Fix For: 1.3.6


When extending MarkupCache to hook inother MarkupLoader for certain components, 
I'd like to be able to delegate a call to clearing the cache to this other 
markup loader.

To be able to do so, I need to have the final removed from MarkupCache.

Use case: http://www.footprint.de/fcc/2008/11/some-wicket-scala/ (To delegate 
the clear to FreeMarker/Velocity e.g.)

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



[jira] Commented: (WICKET-1965) Remove final from MarkupCache#clear()

2008-12-01 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12652146#action_12652146
 ] 

Jan Kriesten commented on WICKET-1965:
--

Timo,

ouch - you're right! I think I was just blind on that eye. :-)

Best regards, --- Jan.

 Remove final from MarkupCache#clear()
 -

 Key: WICKET-1965
 URL: https://issues.apache.org/jira/browse/WICKET-1965
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.4-RC1
Reporter: Jan Kriesten
Priority: Minor
 Fix For: 1.3.6


 When extending MarkupCache to hook inother MarkupLoader for certain 
 components, I'd like to be able to delegate a call to clearing the cache to 
 this other markup loader.
 To be able to do so, I need to have the final removed from MarkupCache.
 Use case: http://www.footprint.de/fcc/2008/11/some-wicket-scala/ (To delegate 
 the clear to FreeMarker/Velocity e.g.)

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



[jira] Closed: (WICKET-1965) Remove final from MarkupCache#clear()

2008-12-01 Thread Jan Kriesten (JIRA)

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

Jan Kriesten closed WICKET-1965.


Resolution: Invalid

 Remove final from MarkupCache#clear()
 -

 Key: WICKET-1965
 URL: https://issues.apache.org/jira/browse/WICKET-1965
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.4-RC1
Reporter: Jan Kriesten
Priority: Minor
 Fix For: 1.3.6


 When extending MarkupCache to hook inother MarkupLoader for certain 
 components, I'd like to be able to delegate a call to clearing the cache to 
 this other markup loader.
 To be able to do so, I need to have the final removed from MarkupCache.
 Use case: http://www.footprint.de/fcc/2008/11/some-wicket-scala/ (To delegate 
 the clear to FreeMarker/Velocity e.g.)

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



[jira] Created: (WICKET-1839) IAjaxIndicatorAware/WicketAjaxIndicatorAppender with AutoCompleteTextField doesn't work

2008-09-17 Thread Jan Kriesten (JIRA)
IAjaxIndicatorAware/WicketAjaxIndicatorAppender with AutoCompleteTextField 
doesn't work
---

 Key: WICKET-1839
 URL: https://issues.apache.org/jira/browse/WICKET-1839
 Project: Wicket
  Issue Type: Bug
  Components: wicket-extensions
Affects Versions: 1.3.4
 Environment: Any
Reporter: Jan Kriesten
 Fix For: 1.3.5


When adding IAjaxIndicatorAware/WicketAjaxIndicatorAppender to an 
AutoCompleteTextField to indicate loading of remote data, the Indicator isn't 
activated.

On a side note: AbstractAutoCompleteBehavior#onBind adds another 
AbstractDefaultAjaxBehavior which is unnecessary since 
AbstractAutoCompleteBehavior extends that already.

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



[jira] Updated: (WICKET-1839) IAjaxIndicatorAware/WicketAjaxIndicatorAppender with AutoCompleteTextField doesn't work

2008-09-17 Thread Jan Kriesten (JIRA)

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

Jan Kriesten updated WICKET-1839:
-

Attachment: indicator-test.tar.gz

Test case.

 IAjaxIndicatorAware/WicketAjaxIndicatorAppender with AutoCompleteTextField 
 doesn't work
 ---

 Key: WICKET-1839
 URL: https://issues.apache.org/jira/browse/WICKET-1839
 Project: Wicket
  Issue Type: Bug
  Components: wicket-extensions
Affects Versions: 1.3.4
 Environment: Any
Reporter: Jan Kriesten
 Fix For: 1.3.5

 Attachments: indicator-test.tar.gz


 When adding IAjaxIndicatorAware/WicketAjaxIndicatorAppender to an 
 AutoCompleteTextField to indicate loading of remote data, the Indicator isn't 
 activated.
 On a side note: AbstractAutoCompleteBehavior#onBind adds another 
 AbstractDefaultAjaxBehavior which is unnecessary since 
 AbstractAutoCompleteBehavior extends that already.

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



[jira] Commented: (WICKET-1773) DiskPageStore-FileNotFoundException

2008-09-04 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12628333#action_12628333
 ] 

Jan Kriesten commented on WICKET-1773:
--


Behavior is reproducable when sessions are invalidated not within wicket but 
e.g. Spring Security.

It turns out that DiskPageStore#unbind calls flushPagesToSaveList() which again 
tries to write to the no longer existing SessionStore.

Just clearing the list of pages instead of trying to write them solves the 
problem:

---8---
public void unbind(String sessionId)
{   
SessionEntry entry = 
(SessionEntry)sessionIdToEntryMap.remove(sessionId);
if (entry != null)
{   
if (isSynchronous())
{   
entry.unbind();
}   
else
{
List pages = getPagesToSaveList(sessionId);
if( pages!=null )
pages.clear();
entry.unbind();
pagesToSaveAll.remove(sessionId);
}
}
}
---8---


 DiskPageStore-FileNotFoundException
 ---

 Key: WICKET-1773
 URL: https://issues.apache.org/jira/browse/WICKET-1773
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.4
Reporter: Jan Kriesten
Assignee: Matej Knopp
 Fix For: 1.3.5


 With the current 1.3-Snapshot, I encounter problems with DiskPageStore:
 ---8---
 09:30:43.151 ERROR [.wicket.protocol.http.pagestore.DiskPageStore] - Error 
 flushing page
 java.lang.RuntimeException: java.io.FileNotFoundException: 
 /usr/local/www/services/local.silberlicht.de/html/WEB-INF/tmp/Silberlicht-filestore/abcgm_hyTIaiqgnqDNmUr/pm-null
  (No such file or directory)
 at 
 org.apache.wicket.protocol.http.pagestore.FileChannelPool.newFileChannel(FileChannelPool.java:104)
 at 
 org.apache.wicket.protocol.http.pagestore.FileChannelPool.getFileChannel(FileChannelPool.java:171)
 at 
 org.apache.wicket.protocol.http.pagestore.DiskPageStore$SessionEntry.savePage(DiskPageStore.java:241)
 at 
 org.apache.wicket.protocol.http.pagestore.DiskPageStore.flushPagesToSaveList(DiskPageStore.java:891)
 at 
 org.apache.wicket.protocol.http.pagestore.DiskPageStore$PageSavingThread.run(DiskPageStore.java:961)
 at java.lang.Thread.run(Thread.java:613)
 Caused by: java.io.FileNotFoundException: 
 /usr/local/www/services/local.silberlicht.de/html/WEB-INF/tmp/Silberlicht-filestore/abcgm_hyTIaiqgnqDNmUr/pm-null
  (No such file or directory)
 at java.io.RandomAccessFile.open(Native Method)
 at java.io.RandomAccessFile.init(RandomAccessFile.java:212)
 at 
 org.apache.wicket.protocol.http.pagestore.FileChannelPool.newFileChannel(FileChannelPool.java:99)
 ... 5 common frames omitted
 ---8---
 The path 
 /usr/local/www/services/local.silberlicht.de/html/WEB-INF/tmp/Silberlicht-filestore/
  actually exists and has the proper rights - so creation of temporary 
 directories/files should be possible (actually, the directory was created by 
 the wicket app).

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



[jira] Commented: (WICKET-1560) MarkupFragmentFinder fails on transparent resolvers within Repeaters

2008-07-02 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12610121#action_12610121
 ] 

Jan Kriesten commented on WICKET-1560:
--

If you don't extend AbstractRepeater, in which way does this affect you at all?

 MarkupFragmentFinder fails on transparent resolvers within Repeaters
 

 Key: WICKET-1560
 URL: https://issues.apache.org/jira/browse/WICKET-1560
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3, 1.4-M1
 Environment: Any
Reporter: Jan Kriesten
Assignee: Gerolf Seitz
Priority: Critical
 Fix For: 1.3.4, 1.4-M2


 I extended the AjaxDataTable to be able to add Rows to certain states of the 
 rows. However, MarkupFragmentFinder fails to resolve the code under this 
 condition since it compares with the wrong components in this case:
 Markup:
 wicket:container wicket:id=rows
   tr wicket:id=rowtd wicket:id=cellsspan 
 wicket:id=cell[cell]/span/td/tr
   tr wicket:id=action-row class=actionRow/tr
 /wicket:container
 'row' is the transparent resolver in this case to maintain the hierarchy 
 needed by the DataTable. MarkupFragmentFinder fails in the case of adding a 
 cell to an AjaxRequestTarget. 
 Fix:
 Everything works as expected, when MarkupFragmentFinder checks for the parent 
 being an AbstractRepeater and in the case compares the parent.id with the 
 supplied id (code provided by Gerolf):
 if (elem instanceof ComponentTag)
 {
   ComponentTag tag = (ComponentTag)elem;
   String id = tag.getId();
   if ((id != null)  id.equals(component.getId()))
   {
 // Ok, found it
 return markupStream;
   }
   else   
   {   
 Component parent = component.getParent();   
 if (parent instanceof AbstractRepeater  id != null  
 id.equals(parent.getId()))   
 {   
   return markupStream;   
 }   
   }
 }

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



[jira] Created: (WICKET-1564) filter-restore script-tag isn't xhtml-valid

2008-04-25 Thread Jan Kriesten (JIRA)
filter-restore script-tag isn't xhtml-valid
---

 Key: WICKET-1564
 URL: https://issues.apache.org/jira/browse/WICKET-1564
 Project: Wicket
  Issue Type: Improvement
  Components: wicket-extensions
Affects Versions: 1.3.3
Reporter: Jan Kriesten
Priority: Trivial


The script-tag needs to be modified from script.../script to

script type=text/javascript.../script

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



[jira] Updated: (WICKET-1564) filter-restore script-tag isn't xhtml-valid

2008-04-25 Thread Jan Kriesten (JIRA)

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

Jan Kriesten updated WICKET-1564:
-

Description: 
The script-tag needs to be modified from script.../script to

script type=text/javascript.../script

File: 
org/apache/wicket/extensions/markup/html/repeater/data/table/filter/FilterForm.java

  was:
The script-tag needs to be modified from script.../script to

script type=text/javascript.../script


 filter-restore script-tag isn't xhtml-valid
 ---

 Key: WICKET-1564
 URL: https://issues.apache.org/jira/browse/WICKET-1564
 Project: Wicket
  Issue Type: Improvement
  Components: wicket-extensions
Affects Versions: 1.3.3
Reporter: Jan Kriesten
Priority: Trivial

 The script-tag needs to be modified from script.../script to
 script type=text/javascript.../script
 File: 
 org/apache/wicket/extensions/markup/html/repeater/data/table/filter/FilterForm.java

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



[jira] Created: (WICKET-1560) MarkupFragmentFinder fails on transparent resolvers within Repeaters

2008-04-23 Thread Jan Kriesten (JIRA)
MarkupFragmentFinder fails on transparent resolvers within Repeaters


 Key: WICKET-1560
 URL: https://issues.apache.org/jira/browse/WICKET-1560
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
 Environment: Any
Reporter: Jan Kriesten
Priority: Critical
 Fix For: 1.3.4


I extended the AjaxDataTable to be able to add Rows to certain states of the 
rows. However, MarkupFragmentFinder fails to resolve the code under this 
condition since it compares with the wrong components in this case:

Markup:

wicket:container wicket:id=rows
  tr wicket:id=rowtd wicket:id=cellsspan 
wicket:id=cell[cell]/span/td/tr
  tr wicket:id=action-row class=actionRow/tr
/wicket:container

'row' is the transparent resolver in this case to maintain the hierarchy needed 
by the DataTable. MarkupFragmentFinder fails in the case of adding a cell to an 
AjaxRequestTarget. 

Fix:

Everything works as expected, when MarkupFragmentFinder checks for the parent 
being an AbstractRepeater and in the case compares the parent.id with the 
supplied id (code provided by Gerolf):

if (elem instanceof ComponentTag)
{
  ComponentTag tag = (ComponentTag)elem;
  String id = tag.getId();
  if ((id != null)  id.equals(component.getId()))
  {
// Ok, found it
return markupStream;
  }
  else   
  {   
Component parent = component.getParent();   
if (parent instanceof AbstractRepeater  id != null  
id.equals(parent.getId()))   
{   
  return markupStream;   
}   
  }
}



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



[jira] Commented: (WICKET-1560) MarkupFragmentFinder fails on transparent resolvers within Repeaters

2008-04-23 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12591580#action_12591580
 ] 

Jan Kriesten commented on WICKET-1560:
--

Just realized:

Maybe it doesn't has to do anything with transparent resolvers at all but only 
with nested repeaters and adding cells from the innermost.

 MarkupFragmentFinder fails on transparent resolvers within Repeaters
 

 Key: WICKET-1560
 URL: https://issues.apache.org/jira/browse/WICKET-1560
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
 Environment: Any
Reporter: Jan Kriesten
Priority: Critical
 Fix For: 1.3.4


 I extended the AjaxDataTable to be able to add Rows to certain states of the 
 rows. However, MarkupFragmentFinder fails to resolve the code under this 
 condition since it compares with the wrong components in this case:
 Markup:
 wicket:container wicket:id=rows
   tr wicket:id=rowtd wicket:id=cellsspan 
 wicket:id=cell[cell]/span/td/tr
   tr wicket:id=action-row class=actionRow/tr
 /wicket:container
 'row' is the transparent resolver in this case to maintain the hierarchy 
 needed by the DataTable. MarkupFragmentFinder fails in the case of adding a 
 cell to an AjaxRequestTarget. 
 Fix:
 Everything works as expected, when MarkupFragmentFinder checks for the parent 
 being an AbstractRepeater and in the case compares the parent.id with the 
 supplied id (code provided by Gerolf):
 if (elem instanceof ComponentTag)
 {
   ComponentTag tag = (ComponentTag)elem;
   String id = tag.getId();
   if ((id != null)  id.equals(component.getId()))
   {
 // Ok, found it
 return markupStream;
   }
   else   
   {   
 Component parent = component.getParent();   
 if (parent instanceof AbstractRepeater  id != null  
 id.equals(parent.getId()))   
 {   
   return markupStream;   
 }   
   }
 }

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



[jira] Created: (WICKET-1542) Transparent Resolvers as targets for Ajax requests

2008-04-16 Thread Jan Kriesten (JIRA)
Transparent Resolvers as targets for Ajax requests
--

 Key: WICKET-1542
 URL: https://issues.apache.org/jira/browse/WICKET-1542
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.3.3
 Environment: Any
Reporter: Jan Kriesten


As it is now, using Componants which are transparent resolvers as targets for 
Ajax requests don't lead to actually rerender their Markup-children.

Now I have an urgent implementation issue with this where I want to have two 
table rows with a DataTable for certain rows, so I have to change the Markup 
hierarchy. To not break the functionality of the contained DataGrid, I have to 
use a transparent resolver:

  wicket:container wicket:id=rows
tr wicket:id=rowtd wicket:id=cellsspan 
wicket:id=cell[cell]/span/td/tr
tr wicket:id=action-rowtd wicket:id=action-cellsspan 
wicket:id=action-cell[cell]/span/td/tr
  /wicket:container

'row' would be the transparent resolver in this case - and also a target for 
Ajax requests.

This doesn't work at the moment, since the 'cells' aren't re-rendered on Ajax 
requests. 

Is there another solution to this problem I didn't think of? Or is there 
another solution to have the cells rerendered?


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



[jira] Created: (WICKET-1519) wicket:component : header-contributions don't work atm

2008-04-10 Thread Jan Kriesten (JIRA)
wicket:component : header-contributions don't work atm
--

 Key: WICKET-1519
 URL: https://issues.apache.org/jira/browse/WICKET-1519
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten


Currently, components embedded thru templates using the wicket:component-tag 
seem to be ignored when it comes to header contributions. E.g. 
AjaxTimerBehavior don't work.

It would be nice if wicket:component could be enhanced in that respect.

Best regards, --- Jan.

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



[jira] Created: (WICKET-1501) MarkupCache.putIntoCache doesn't behave correctly!!

2008-04-09 Thread Jan Kriesten (JIRA)
MarkupCache.putIntoCache doesn't behave correctly!!
---

 Key: WICKET-1501
 URL: https://issues.apache.org/jira/browse/WICKET-1501
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten
Priority: Critical


'putIntoCache' checks for the locationString not to be null instead of the 
cacheKey!

This way you always get old markup returned instead of the freshly supplied 
markup which shouldn't be cached at all.

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



[jira] Commented: (WICKET-1501) MarkupCache.putIntoCache doesn't behave correctly!!

2008-04-09 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12587093#action_12587093
 ] 

Jan Kriesten commented on WICKET-1501:
--

I don't actually understand why the markup is loaded from the markupCache in 
putIntoCache at all - it's already provided as a method argument...

 MarkupCache.putIntoCache doesn't behave correctly!!
 ---

 Key: WICKET-1501
 URL: https://issues.apache.org/jira/browse/WICKET-1501
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten
Priority: Critical

 'putIntoCache' checks for the locationString not to be null instead of the 
 cacheKey!
 This way you always get old markup returned instead of the freshly supplied 
 markup which shouldn't be cached at all.

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



[jira] Commented: (WICKET-1501) MarkupCache.putIntoCache doesn't behave correctly!!

2008-04-09 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12587155#action_12587155
 ] 

Jan Kriesten commented on WICKET-1501:
--

Johan -

I return 'null' for the cacheKey of my template. That's fine. The Markup isn't 
cached _under_this_ key.

The problem arises because wicket caches the Markup in putIntoCache also under 
the locationString - which has nothing to do with the cacheKey. So, markup is 
cached under two keys by wicket, which leads to the problem. 

Regards, --- Jan.

 MarkupCache.putIntoCache doesn't behave correctly!!
 ---

 Key: WICKET-1501
 URL: https://issues.apache.org/jira/browse/WICKET-1501
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten
Assignee: Juergen Donnerstag
Priority: Critical

 'putIntoCache' checks for the locationString not to be null instead of the 
 cacheKey!
 This way you always get old markup returned instead of the freshly supplied 
 markup which shouldn't be cached at all.

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



[jira] Commented: (WICKET-1501) MarkupCache.putIntoCache doesn't behave correctly!!

2008-04-09 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12587163#action_12587163
 ] 

Jan Kriesten commented on WICKET-1501:
--

hi johan,

huh?? i have no cache key for the template - it just shall not be handled by 
wicket's cache system (it's already cached by freemarker and changes every 
request)

so - not having a cache key means i can't lookup the locationString in the 
first map, right?

regards, --- jan.

 MarkupCache.putIntoCache doesn't behave correctly!!
 ---

 Key: WICKET-1501
 URL: https://issues.apache.org/jira/browse/WICKET-1501
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten
Assignee: Juergen Donnerstag
Priority: Critical

 'putIntoCache' checks for the locationString not to be null instead of the 
 cacheKey!
 This way you always get old markup returned instead of the freshly supplied 
 markup which shouldn't be cached at all.

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



[jira] Commented: (WICKET-1501) MarkupCache.putIntoCache doesn't behave correctly!!

2008-04-09 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12587179#action_12587179
 ] 

Jan Kriesten commented on WICKET-1501:
--

That's what I was trying to say:

Even though cacheKey is 'null' - the markup is still cached under 
locationString.

Since the resulting Markup is always returned from 'putIntoCache' - the 
originally updated markup in the parameter to 'putIntoCache' is thrown away and 
replaced by the old cached one...

Regards, --- Jan.

 MarkupCache.putIntoCache doesn't behave correctly!!
 ---

 Key: WICKET-1501
 URL: https://issues.apache.org/jira/browse/WICKET-1501
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten
Assignee: Juergen Donnerstag
Priority: Critical

 'putIntoCache' checks for the locationString not to be null instead of the 
 cacheKey!
 This way you always get old markup returned instead of the freshly supplied 
 markup which shouldn't be cached at all.

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



[jira] Updated: (WICKET-1495) Ajax, MulitlineLabel, AjaxEditableMultiLineLabel

2008-04-08 Thread Jan Kriesten (JIRA)

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

Jan Kriesten updated WICKET-1495:
-

Attachment: tmp.txt

Wicket-Debug for MultiLineLabel + Label

 Ajax, MulitlineLabel, AjaxEditableMultiLineLabel
 

 Key: WICKET-1495
 URL: https://issues.apache.org/jira/browse/WICKET-1495
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten
 Attachments: tmp.txt


 I encountered the following problems when using AjaxEditableMultiLineLabel:
 a) MultiLineLabels aren't rendered with Internet Explorer 7 when content 
 contains something like pre?xml... but give a runtime error. The same 
 code works fine when using a 'normal' label, though! I'll attach a file 
 showing the Wicket-Debug for both cases.
 b) AEML doesn't show anything on empty string models

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



[jira] Commented: (WICKET-1495) Ajax, MulitlineLabel, AjaxEditableMultiLineLabel

2008-04-08 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1495?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12586773#action_12586773
 ] 

Jan Kriesten commented on WICKET-1495:
--

Wicket-Debug had a bug displaying entities as  etc. Matej resolved this.

IE seems to have a problem rendering br/ within pre /pre on 
Ajax-Requests. Resolved this issue using Label instead of MultiLineLabel in 
this case.

 Ajax, MulitlineLabel, AjaxEditableMultiLineLabel
 

 Key: WICKET-1495
 URL: https://issues.apache.org/jira/browse/WICKET-1495
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
Reporter: Jan Kriesten
 Attachments: tmp.txt


 I encountered the following problems when using AjaxEditableMultiLineLabel:
 a) MultiLineLabels aren't rendered with Internet Explorer 7 when content 
 contains something like pre?xml... but give a runtime error. The same 
 code works fine when using a 'normal' label, though! I'll attach a file 
 showing the Wicket-Debug for both cases.
 b) AEML doesn't show anything on empty string models

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



[jira] Created: (WICKET-1499) AjaxEditableMultiLineLabel + race condition /

2008-04-08 Thread Jan Kriesten (JIRA)
AjaxEditableMultiLineLabel + race condition / 
--

 Key: WICKET-1499
 URL: https://issues.apache.org/jira/browse/WICKET-1499
 Project: Wicket
  Issue Type: Bug
  Components: wicket-extensions
Affects Versions: 1.3.3
Reporter: Jan Kriesten



There are two issues concerning AjaxEditableMultiLineLabel:

a) Race condition cancel editing

Using 'Esc' to cancel editing might lead to two events to be fired: first 
onKeypress is executed - which leads to bluring the component in Opera. This 
results in the onblur-event firing and the input is submitted!

Changing the code in onComponentTag to

protected void onComponentTag(ComponentTag tag)
{   
super.onComponentTag(tag);
final String saveCall = {wicketAjaxGet(' + 
getCallbackUrl() +

save=true'+this.name+'='+wicketEncode(this.value)); return false;};

final String cancelCall = {wicketAjaxGet(' + 
getCallbackUrl() +
save=false'); this.onblur=''; return 
false;};

final String keypress = var 
kc=wicketKeyCode(event); if (kc==27)  + cancelCall +
; ;

tag.put(onblur, saveCall);
tag.put(onkeypress, keypress);
}

stops onblur being fired.

This might also apply for AjaxEditableLabel - though I haven't seen this 
happening there yet.


b) Displaying defaultNullLabel on empty String Model:

Should behave like AjaxEditableLabel, i.e.:

protected void onComponentTagBody(MarkupStream markupStream, 
ComponentTag openTag)
{
Object modelObject = getModelObject();
if (modelObject == null || .equals(modelObject))
{
replaceComponentTagBody(markupStream, openTag, 
defaultNullLabel());
}
else
{
super.onComponentTagBody(markupStream, openTag);
}
} 



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



[jira] Created: (WICKET-1469) New Wicket tag 'wicket:for'

2008-04-01 Thread Jan Kriesten (JIRA)
New Wicket tag 'wicket:for'
---

 Key: WICKET-1469
 URL: https://issues.apache.org/jira/browse/WICKET-1469
 Project: Wicket
  Issue Type: New Feature
  Components: wicket
Affects Versions: 1.3.2
Reporter: Jan Kriesten
Priority: Minor


This often happens during my daily work:

You create a form with labels and corresponding input fields. As it is now, you 
have to bind all those Labels and FormComponents together with some 
boilerplate code within Java.

I'd like to suggest the following enhancement Wicket tag:

label wicket:for=username wicket:messge=keydefault message/label

where wicket:for contains the referenced wicket:id


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



[jira] Commented: (WICKET-1175) IDataProvider-Overflow with size()

2007-11-21 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12544422
 ] 

Jan Kriesten commented on WICKET-1175:
--

hi johan,

i don't think it's a question of sense, but to be able to handle such 
situations. as stated elsewhere - jpa also delivers longs, so currently you 
strip those down to int, too, which is not a nice thing at all imho.

regards, --- jan.

 IDataProvider-Overflow with size()
 --

 Key: WICKET-1175
 URL: https://issues.apache.org/jira/browse/WICKET-1175
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.3.0-rc1
 Environment: any
Reporter: Jan Kriesten
Assignee: Igor Vaynberg

 Hi,
 I get an Integer-overflow with my Dataprovider (yeah, there are a couple of 
 entries in the database). Is there a reason why size() and iterator( first, 
 count ) are limited to Integer?
 Regards, --- Jan.

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



[jira] Updated: (WICKET-1098) AjaxEditableLabel: add method to override edit-event

2007-11-14 Thread Jan Kriesten (JIRA)

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

Jan Kriesten updated WICKET-1098:
-

Attachment: ael.patch

Hi Johan,

thanks for looking into it. I attached a small patch which should be sufficient.

Best regards, --- Jan.

 AjaxEditableLabel: add method to override edit-event
 

 Key: WICKET-1098
 URL: https://issues.apache.org/jira/browse/WICKET-1098
 Project: Wicket
  Issue Type: Improvement
  Components: wicket-extensions
Affects Versions: 1.3.0-beta4
 Environment: any
Reporter: Jan Kriesten
Assignee: Johan Compagner
Priority: Minor
 Fix For: 1.3.0-rc2

 Attachments: ael.patch


 Currently, as event 'onclick' is hardcoded in AjaxEditableLabel. To change 
 this behavior, a bunch of logic has to be copied. It would be nice, if there 
 was a method
 newLabelAjaxBehavior(String event)
 which could be easily overriden.
 Might apply to other AjaxEdit-Components as well.

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



[jira] Created: (WICKET-1131) AjaxEditableLabel: defaultNullLabel() should really be a defaultNullorEmptyLabel()

2007-11-05 Thread Jan Kriesten (JIRA)
AjaxEditableLabel: defaultNullLabel() should really be a 
defaultNullorEmptyLabel()
--

 Key: WICKET-1131
 URL: https://issues.apache.org/jira/browse/WICKET-1131
 Project: Wicket
  Issue Type: Bug
Affects Versions: 1.3.0-beta4
 Environment: any
Reporter: Jan Kriesten
 Fix For: 1.3.0-rc1


defaultNullLabel() only inserts '...' when the model is NULL, not when the 
model contains an empty String.

So right now it's impossible to get empty Strings edited with this component.

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



[jira] Created: (WICKET-1097) AjaxEditableLabel: Model change events not propagated from Editor

2007-10-24 Thread Jan Kriesten (JIRA)
AjaxEditableLabel: Model change events not propagated from Editor
-

 Key: WICKET-1097
 URL: https://issues.apache.org/jira/browse/WICKET-1097
 Project: Wicket
  Issue Type: Improvement
  Components: wicket-extensions
Affects Versions: 1.3.0-beta4
 Environment: any
Reporter: Jan Kriesten


Model changes are not propagated from the Editor to the AjaxEditableLabel, so 
overriding onModelChanging/onModelChanged doesn't work as one might expect. The 
implementation should be changed to something like this (code sample by Gerolf):

---
protected FormComponent newEditor(MarkupContainer parent, String componentId, 
IModel model)   
{   
TextField editor = new TextField(componentId, model)   
{   
private static final long serialVersionUID = 1L;   
  
protected void onModelChanging()   
{   
super.onModelChanging();   
AjaxEditableLabel.this.onModelChanging();   
}   
  
protected void onModelChanged()   
{   
super.onModelChanged();   
AjaxEditableLabel.this.onModelChanged();   
}   
};   
editor.setOutputMarkupId(true);   
editor.setVisible(false);   
editor.add(new EditorAjaxBehavior());   
return editor;   
}
---

The same issue might apply to the other inline-editors.


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



[jira] Created: (WICKET-1080) StringResourceModel.toString() misbehavior

2007-10-17 Thread Jan Kriesten (JIRA)
StringResourceModel.toString() misbehavior
--

 Key: WICKET-1080
 URL: https://issues.apache.org/jira/browse/WICKET-1080
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.0-beta4
 Environment: any
Reporter: Jan Kriesten


StringResourceModel.toString() doesn't behave like the API documentation states 
any more.

It returns a lot of debug information instead of the the getString()-result as 
stated in the docs.



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



[jira] Created: (WICKET-1069) RFE: DataTable colgroup

2007-10-13 Thread Jan Kriesten (JIRA)
RFE: DataTable  colgroup
--

 Key: WICKET-1069
 URL: https://issues.apache.org/jira/browse/WICKET-1069
 Project: Wicket
  Issue Type: Improvement
  Components: wicket-extensions
 Environment: any
Reporter: Jan Kriesten



Hi!

I want to suggest an enhancement for the DataTable extension.

Right now it's only possible to style columns (width, alignment, color etc.) 
via stylesheets. This has some not so nice implications IMHO:

- You have to extend the IStyledColumn to return the proper CSS class.
- You have to add a bunch of CSS classes to get the proper styles.
- You have redundancy in HTML output - which isn't necessary.

This is actually what colgroupcol/colgroup is meant to solve.

My suggestion:

Adding a 'addColGroup( ColGroup cg )'-feature, which actually lets you add x 
ColGroups with n Cols on which you can use AttributeModifiers to have your 
styles/classes/width-Attributes added.

A quick implementation would be this:

---[ColGroup.java]--
import java.util.List;

import org.apache.wicket.behavior.IBehavior;
import 
org.apache.wicket.extensions.markup.html.repeater.data.table.AbstractToolbar;
import org.apache.wicket.extensions.markup.html.repeater.data.table.DataTable;
import org.apache.wicket.markup.html.WebMarkupContainer;
import org.apache.wicket.markup.repeater.RepeatingView;

public class ColGroup
extends AbstractToolbar
{
  private static final long serialVersionUID = 1L;

  private int numCols;
  private Col[] cols;

  public ColGroup( DataTable datatable, int numCols )
  {
super( datatable );
this.numCols = numCols;
  }

  public void onBeforeRender()
  {
if( !hasBeenRendered() )
{
  WebMarkupContainer colgroup = new WebMarkupContainer( colgroup );
  for( IBehavior b : (ListIBehavior) getBehaviors() )
colgroup.add( b );
  setRenderBodyOnly( true );
  removeAll();
  add( colgroup );

  RepeatingView colgroupCols = new RepeatingView( elements );
  colgroup.add( colgroupCols );

  for( Col column : getCols() )
  {
WebMarkupContainer item = new WebMarkupContainer( 
colgroupCols.newChildId() );
colgroupCols.add( item );
item.add( column );
  }

}
super.onBeforeRender();
  }

  public final Col[] getCols( )
  {
if( cols == null )
{
  cols = new Col[numCols];
  for( int i = 0; i  numCols; i++ )
cols[i] = new Col();
}
return(cols);
  }

  public final class Col
  extends WebMarkupContainer
  {
private static final long serialVersionUID = 1L;

private Col()
{
  super( col );
}
  }

}
---[/ColGroup.java]--

---[ColGroup.html]--
  wicket:panel
colgroup wicket:id=colgroup
  wicket:container wicket:id=elementscol 
wicket:id=col//wicket:container
/colgroup
  /wicket:panel
---[/ColGroup.html]--


Usage:

ColGroup colgroup = new ColGroup( datatable, 4 );
colgroup.add( new SimpleAttributeModifier( style, border: solid 1px green; 
) );
Col[] cols = colgroup.getCols();
cols[0].add( new SimpleAttributeModifier( width, 10% ) );
cols[1].add( new SimpleAttributeModifier( width, 35% ) );
cols[2].add( new SimpleAttributeModifier( width, 45% ) );
cols[3].add( new SimpleAttributeModifier( width, 10% ) );
datatable.addColGroup( colgroup );


Any thoughts?

Regards, --- Jan.




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



[jira] Created: (WICKET-1006) Mount vs setPageExpiredErrorPage

2007-09-25 Thread Jan Kriesten (JIRA)
Mount vs setPageExpiredErrorPage


 Key: WICKET-1006
 URL: https://issues.apache.org/jira/browse/WICKET-1006
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.3.0-beta3
 Environment: n/a
Reporter: Jan Kriesten
Priority: Minor


I have the following simple scenario:

app.mount( new IndexedParamUrlCodingStrategy( login, LoginPage.class ) );
app.getApplicationSettings().setPageExpiredErrorPage( LoginPage.class );

If I call the application, I get directed to the url

http://appurl/login

fine. When the Session times out, I get redirected to the LoginPage again, but
this time not to the above url but to

http://appurl/?wicket:interface=:2

Couldn't the PageExpiredErrorPage be checked for Mounts?


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



[jira] Created: (WICKET-930) Wrap Guice-Injector with proxying for Objects

2007-09-05 Thread Jan Kriesten (JIRA)
Wrap Guice-Injector with proxying for Objects
-

 Key: WICKET-930
 URL: https://issues.apache.org/jira/browse/WICKET-930
 Project: Wicket
  Issue Type: New Feature
  Components: wicket-guice
Reporter: Jan Kriesten
Priority: Minor


Since the GuiceComponentInjector already has proxying support build in, I'd 
like to use this to inject other Objects besides from Components.

A function like

guiceComponentInjector.inject( Object xy )

would be great!



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



[jira] Commented: (WICKET-842) html wicket:id=html is broken again...

2007-08-14 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-842?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12519616
 ] 

Jan Kriesten commented on WICKET-842:
-

IMHO the problem is, that you don't always know what the parent defines. So 
when you extend a page class, you should always be able to add components 
without knowing what structure has been defined yet.

html (and maybe head and body, too) are special tags in that matter. They 
are taken as given when not specifically defined with a wicket:id. You 
automatically add to them - and they're generated for you if not defined. No 
other tags are generated by wicket, so in that respect they're outstanding 
anyway.



 html wicket:id=html is broken again...
 --

 Key: WICKET-842
 URL: https://issues.apache.org/jira/browse/WICKET-842
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.0-beta3
 Environment: JDK 1.6 / OpenSuSE 10.2
Reporter: Jan Kriesten
 Attachments: test.tar.gz


 hi,
 some changes in latest trunk broke adding a wicket:id to html again!  :-( 
 actually, the wicket:id in html works. but extending such a basepage and 
 then
 adding a component to it doesn't work any more! the component hierarchy 
 doesn't
 seem to be resolved correctly.
 test case is attached.
 regards, --- jan.

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



[jira] Commented: (WICKET-842) html wicket:id=html is broken again...

2007-08-14 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-842?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12519867
 ] 

Jan Kriesten commented on WICKET-842:
-

Igor,

your example is very different from this case. You explicitely build a 
hierarchy with the div - the html would live without it, too.

But you always have to have a html and a body, so these are entry points 
for headers and markup and so are created by wicket if not defined! And as 
these are defined as a container by default (else you couldn't add a header or 
elements to the page at all), there is actually not really a change in the 
hierarchy if you add an id to html.

But at last, it's your decision, how wicket's to behave. :-) Case closed.

 html wicket:id=html is broken again...
 --

 Key: WICKET-842
 URL: https://issues.apache.org/jira/browse/WICKET-842
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.0-beta3
 Environment: JDK 1.6 / OpenSuSE 10.2
Reporter: Jan Kriesten
 Attachments: test.tar.gz


 hi,
 some changes in latest trunk broke adding a wicket:id to html again!  :-( 
 actually, the wicket:id in html works. but extending such a basepage and 
 then
 adding a component to it doesn't work any more! the component hierarchy 
 doesn't
 seem to be resolved correctly.
 test case is attached.
 regards, --- jan.

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



[jira] Commented: (WICKET-842) html wicket:id=html is broken again...

2007-08-13 Thread Jan Kriesten (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-842?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12519593
 ] 

Jan Kriesten commented on WICKET-842:
-

Having html behaving like any other tag is IMHO fine.

But I think it a problem that one has to know what component hierarchy has been 
set up in base Page classes (and which has been completely handled by these). 
If I change the base Page class to html wicket:id or remove that again, the 
whole application based on that class shouldn't break.

Shouldn't html be a transparent resolver by default?

Also, can html wicket:id=xy be overwritten by child pages? What happens, if 
the child page has an html wicket:id=yz?

Regards, --- Jan.

 html wicket:id=html is broken again...
 --

 Key: WICKET-842
 URL: https://issues.apache.org/jira/browse/WICKET-842
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.0-beta3
 Environment: JDK 1.6 / OpenSuSE 10.2
Reporter: Jan Kriesten
 Attachments: test.tar.gz


 hi,
 some changes in latest trunk broke adding a wicket:id to html again!  :-( 
 actually, the wicket:id in html works. but extending such a basepage and 
 then
 adding a component to it doesn't work any more! the component hierarchy 
 doesn't
 seem to be resolved correctly.
 test case is attached.
 regards, --- jan.

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