dev@javaserverfaces.java.net

Re: Please do the code review for jsf bugs:3574,3694, 3695

From: zhijun Ren <ren.zhijun_at_oracle.com>
Date: Tue, 03 Feb 2015 17:00:03 +0800

Include your oracle mail in case that you miss the mails from dev mail.

On 2/3/15, 14:47, zhijun Ren wrote:
> Hi Ed and Manfred,
>
> For the convenience of the remaining flow issues, I have committed
> the code into trunk and set #3694 and #3695 as resolved.
>
> /rzhijun-mac:trunk zhijun$ svn commit --file
> changebundle_zhijun_for_3695and3694and3574_on150129.txt --username
> ren.zhijun.oracle
> Sending
> jsf-ri/src/main/java/com/sun/faces/application/NavigationHandlerImpl.java
> Sending
> test/javaee6web/flowTraversalCombinations/src/test/java/com/sun/faces/test/javaee6web/flowtraversalcombinations/ReturnNaviToOtherNodesIT.java
> Sending
> test/javaee6web/flowTraversalCombinations/src/test/java/com/sun/faces/test/javaee6web/flowtraversalcombinations/SwitchNaviToOtherNodesIT.java
> Transmitting file data ...
> Committed revision 14304./
>
> I will send out the code review request for #3475 and #3476 soon.
>
> Let 3574 be opened for porting back to 2.2.8, do you agree?
>
> BR,
> Zhijun
>
>
> On 1/31/15, 2:35, zhijun Ren wrote:
>> Ed, just a reminder in case of that you miss the mail.
>>
>> On 1/29/15, 14:30, zhijun Ren wrote:
>>> All Hudson tests against WLS also have been done locally without any
>>> of them failed, thanks.
>>>
>>> On 1/29/15, 13:50, zhijun Ren wrote:
>>>> Hi Ed and Manfred,
>>>>
>>>> Please help to do the review for the three bugs, change bundle
>>>> attached here for your convenience.
>>>>
>>>> Note that besides fixing the bugs, I have run all Hudson tests
>>>> against GF and didn't fail any of them.
>>>>
>>>> During the working of these faces flow bugs, I really think the
>>>> current design is not good, e.g., code duplication, implicit
>>>> recursive call, many flag variables and if statements, bad
>>>> readability, etc.. , looking forward to refactor by a state machine.
>>>>
>>>> BR,
>>>> Zhijun