Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.6.0-b1
    • Component/s: Plugins
    • Labels:
      None

      Description

      continue statements that are within a switch will raise a warning in php 7.3 because it acts as a break.

      The HTMLPurifier library uses two continue statements in a switch. They function correctly as a break in this context, but they really should be continue 2 instead. It will generate a warning in php 7.3 otherwise.

      Affects 2 locations in:

      • /lib/IDS/vendors/htmlpurifier/HTMLPurifier/ChildDef/Table.php

      Update the library if possible.

        Issue Links

          Activity

          tyson Tyson Phillips (Inactive) created issue -
          tyson Tyson Phillips (Inactive) made changes -
          Field Original Value New Value
          Rank Ranked higher
          tyson Tyson Phillips (Inactive) made changes -
          Link This issue relates to CORE-3102 [ CORE-3102 ]
          tyson Tyson Phillips (Inactive) made changes -
          Description _continue_ statements that are within a _switch_ will raise a warning in php 7.3 because it acts as a _break_.

          The HTMLPurifier library uses two _continue_ statements in a switch. They function correctly as a break in this context, but they really should be _continue 2_ instead. It will generate a warning in php 7.3 otherwise.

          Update the library if possible.
          _continue_ statements that are within a _switch_ will raise a warning in php 7.3 because it acts as a _break_.

          The HTMLPurifier library uses two _continue_ statements in a switch. They function correctly as a break in this context, but they really should be _continue 2_ instead. It will generate a warning in php 7.3 otherwise.

          Affects 2 locations in:
          * /lib/IDS/vendors/htmlpurifier/HTMLPurifier/ChildDef/Table.php

          Update the library if possible.
          tyson Tyson Phillips (Inactive) made changes -
          Sprint 4.6.0 Sprint 6 [ 81 ]
          tyson Tyson Phillips (Inactive) made changes -
          Rank Ranked higher
          tyson Tyson Phillips (Inactive) made changes -
          Assignee Jonathan Reissmueller [ jonathan ]
          tyson Tyson Phillips (Inactive) made changes -
          Sprint 4.6.0 Sprint 6 [ 81 ] 4.6.0 Sprint 6, 4.6.0 Sprint 7 [ 81, 85 ]
          tyson Tyson Phillips (Inactive) made changes -
          Rank Ranked higher
          Automated transition triggered when Jonathan Reissmueller created a branch in Stash -
          Status Open [ 1 ] In Progress [ 3 ]
          jonathan Jonathan Reissmueller made changes -
          Remaining Estimate 0 minutes [ 0 ]
          Time Spent 53 minutes [ 3180 ]
          Worklog Id 12193 [ 12193 ]
          jonathan Jonathan Reissmueller made changes -
          Time Spent 53 minutes [ 3180 ] 3 hours, 53 minutes [ 13980 ]
          Worklog Id 12193 [ 12193 ]
          Automated transition triggered when Jonathan Reissmueller created pull request #6 in Stash -
          Status In Progress [ 3 ] In Review [ 5 ]
          Resolution Fixed [ 1 ]
          Automated transition triggered when Tyson Phillips (Inactive) merged pull request #6 in Stash -
          Status In Review [ 5 ] Closed [ 6 ]
          tyson Tyson Phillips (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          tyson Tyson Phillips (Inactive) made changes -
          Fix Version/s 4.6.0-b1 [ 11117 ]
          tyson Tyson Phillips (Inactive) made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]

            People

            • Assignee:
              jonathan Jonathan Reissmueller
              Reporter:
              tyson Tyson Phillips (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                28/May/19

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 3 hours, 53 minutes
                3h 53m

                  Agile