[jira] Commented: (MYFACES-983) Unnecessary check for UIForm component in ScheduleDetailedDayRenderer

["Bruno Aranda (JIRA)" <de...@myfaces.apache.org>]

    [ http://issues.apache.org/jira/browse/MYFACES-983?page=comments#action_12361515 ] 

Bruno Aranda commented on MYFACES-983:
--------------------------------------

Maybe we could check if the schedule is readonly before the null check?

> Unnecessary check for UIForm component in ScheduleDetailedDayRenderer
> ---------------------------------------------------------------------
>
>          Key: MYFACES-983
>          URL: http://issues.apache.org/jira/browse/MYFACES-983
>      Project: MyFaces
>         Type: Bug
>     Versions: Nightly
>     Reporter: Peter Mahoney
>     Priority: Minor

>
> ScheduleDetailedDayRenderer includes the lines:
>         UIForm parentForm = getParentForm(schedule);
>         if (parentForm == null)
>         {
>             throw new NullPointerException("No parent UIForm found");
>         }
> However, if the schedule is readonly then the form component is not required e.g. on a printable page.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira