Struts 2
  1. Struts 2
  2. WW-2498

Avoid NPE when parseContent attribute is missing on dojo tags/context

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1
    • Fix Version/s: 2.1.1
    • Component/s: Plugin - Dojo Tags
    • Labels:
      None
    • Flags:
      Patch

      Description

      If the Head.PARSE_CONTENT is not available in the context the relevant Dojo tag throws a NPE.
      Undoubtedly users will frequently forget to set parseContent on the page's head tag, or on the respective widget tag so this patch provides default handling for the case where parseContent is null.

      The objective of this patch is to reduce the potential for support queries regarding the NPE within struts 2.

      The patch is the equivalent to WW-2398 only the affected code is repeated in Tree, TabbedPanel, Autocompleter, Submit, TreeNode, DateTimePicker and TextArea

      See also dev discussion: http://www.mail-archive.com/dev@struts.apache.org/msg30811.html

        Issue Links

          Activity

          Jeromy Evans created issue -
          Jeromy Evans made changes -
          Field Original Value New Value
          Attachment dojoPluginParseContent.patch [ 13712 ]
          Jeromy Evans made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Jeromy Evans made changes -
          Link This issue is duplicated by WW-2533 [ WW-2533 ]
          Jeromy Evans made changes -
          Link This issue is duplicated by WW-2398 [ WW-2398 ]
          Antonio Petrelli made changes -
          Workflow Struts - editable closed status [ 44930 ] Struts - editable closed status (temporary) [ 48822 ]
          Antonio Petrelli made changes -
          Workflow Struts - editable closed status (temporary) [ 48822 ] Struts - editable closed status [ 51962 ]
          Jeff Turner made changes -
          Project Import Mon Feb 01 01:17:42 UTC 2010 [ 1264987062082 ]
          Lukasz Lenart made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Jeromy Evans
              Reporter:
              Jeromy Evans
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development