Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-2828

cPanel: "count" on null value causes notice in php 7.2

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 4.4.0-b1
        • Component/s: None
        • Labels:
          None

          Description

          See https://www.blesta.com/forums/index.php?/topic/11928-php-72-cpanel-module/

          count(): Parameter must be an array or an object that implements Countable on line 72 in /components/modules/cpanel/views/default/add_row.pdt

          php 7.2 now raises an error if passing a non-Countable parameter to count()

            Activity

            Ascending order - Click to sort in descending order
            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 -
            Sprint 4.4.0 Sprint 4 [ 71 ]
            tyson Tyson Phillips (Inactive) made changes -
            Rank Ranked higher
            tyson Tyson Phillips (Inactive) made changes -
            Security Private [ 10000 ]
            tyson Tyson Phillips (Inactive) made changes -
            Description See https://www.blesta.com/forums/index.php?/topic/11928-php-72-cpanel-module/


            php 7.2 now raises an error if passing a non-Countable parameter to _count()_             		See https://www.blesta.com/forums/index.php?/topic/11928-php-72-cpanel-module/

            {quote}
            count(): Parameter must be an array or an object that implements Countable on line 72 in /components/modules/cpanel/views/default/add_row.pdt
            {quote}

            php 7.2 now raises an error if passing a non-Countable parameter to _count()_
            Automated transition triggered when Tyson Phillips (Inactive) created a branch in Stash -
            Status Open [ 1 ] In Progress [ 3 ]
            Automated transition triggered when Tyson Phillips (Inactive) created pull request #14 in Stash -
            Status In Progress [ 3 ] In Review [ 5 ]
            Resolution Fixed [ 1 ]
            tyson Tyson Phillips (Inactive) made changes -
            Remaining Estimate 0 minutes [ 0 ]
            Time Spent 19 minutes [ 1140 ]
            Worklog Id 11493 [ 11493 ]
            Automated transition triggered when Tyson Phillips (Inactive) merged pull request #14 in Stash -
            Status In Review [ 5 ] Closed [ 6 ]

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  22/Oct/18

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 19 minutes
                  19m

                    Agile