[jira] Commented: (MYFACES-509) StateManager.saveSerializedView must throw an IllegalStateException when View contains duplicate IDs

2005-09-07 Thread Martin Marinschek (JIRA)
[ http://issues.apache.org/jira/browse/MYFACES-509?page=comments#action_12322829 ] Martin Marinschek commented on MYFACES-509: --- Ok, cool! good that you took the pain to implement this ;) just commit it, I am reassigning this issue to you then.

[jira] Commented: (MYFACES-509) StateManager.saveSerializedView must throw an IllegalStateException when View contains duplicate IDs

2005-09-07 Thread Bruno Aranda (JIRA)
[ http://issues.apache.org/jira/browse/MYFACES-509?page=comments#action_12322865 ] Bruno Aranda commented on MYFACES-509: -- As reported in MYFACES-528, now some examples do not work ok. StateManager.saveSerializedView must throw an

[jira] Commented: (MYFACES-509) StateManager.saveSerializedView must throw an IllegalStateException when View contains duplicate IDs

2005-09-07 Thread Martin Marinschek (JIRA)
[ http://issues.apache.org/jira/browse/MYFACES-509?page=comments#action_12322876 ] Martin Marinschek commented on MYFACES-509: --- Well, this is good! This means we have found a major source of incompatibilities left with regard to the RI, if we

[jira] Commented: (MYFACES-509) StateManager.saveSerializedView must throw an IllegalStateException when View contains duplicate IDs

2005-09-06 Thread Oliver Rossmueller (JIRA)
[ http://issues.apache.org/jira/browse/MYFACES-509?page=comments#action_12322794 ] Oliver Rossmueller commented on MYFACES-509: r279199: Changed the exception thrown by checkForDuplicateIds to IllegalStateException

[jira] Commented: (MYFACES-509) StateManager.saveSerializedView must throw an IllegalStateException when View contains duplicate IDs

2005-09-06 Thread Martin Marinschek (JIRA)
[ http://issues.apache.org/jira/browse/MYFACES-509?page=comments#action_12322804 ] Martin Marinschek commented on MYFACES-509: --- We ought to check for clientIds, not ids. What was wrong was that a clientId of null was not accepted - of course it