http://forge.typo3.org/http://forge.typo3.org/themes/typo3_forge/favicon/favicon.png?17058661692017-08-28T12:04:36ZTYPO3 ForgeTYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3388412017-08-28T12:04:36ZAlexander Opitzopitz.alexander@googlemail.com
<ul><li><strong>PHP Version</strong> set to <i>7.2</i></li></ul> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3388562017-08-28T18:43:10ZAlexander Opitzopitz.alexander@googlemail.com
<ul></ul><p><a class="external" href="https://github.com/TYPO3/Fluid/pull/296">https://github.com/TYPO3/Fluid/pull/296</a></p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3389252017-08-30T13:51:49ZFaton Halitifha@systime.dk
<ul></ul><p>Faced the same with typo3 8.7 and php v7.1</p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3396882017-09-07T20:44:07ZClaus Dueclaus@phpmind.net
<ul></ul><p>You can see the source of the ViewHelper here: <a class="external" href="https://github.com/TYPO3/Fluid/blob/master/src/ViewHelpers/CountViewHelper.php">https://github.com/TYPO3/Fluid/blob/master/src/ViewHelpers/CountViewHelper.php</a></p>
<ul>
<li>The exception is quite intentional and the case you have should have caused PHP warnings before the exception was introduced</li>
<li>The VH is aware of NULL and returns 0 for that case</li>
<li>Which leads me to believe the data you passed here may in fact neither be an array nor countable in which case the error is to be expected?</li>
</ul> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3531252017-12-11T10:27:01ZAlexander Opitzopitz.alexander@googlemail.com
<ul></ul><p>@Claus</p>
<p>As you didn't responded to me on Slack.</p>
<p>This issue is fixed for master (As you can see I added the pull request), but we need a backport for the TYPO3 8 LTS. <br />Also I like that <a class="external" href="https://github.com/TYPO3/Fluid/pull/283">https://github.com/TYPO3/Fluid/pull/283</a> gets somehow backported so we can use it in TYPO3 8 LTS.</p>
<p>TYPO3 8 LTS was released with fluid 2.3.1 and we should try to stay in a line for every LTS. I see that there was a 2.3.4 release, but this do not include the PHP 7.2 fix (nor the one above).</p>
<p>So how we can resolve this here?</p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3534682017-12-15T10:54:25ZSusanne Moogsusanne.moog@typo3.org
<ul></ul><p>For background information: What it would mean to update to 2.4:<br /><a class="external" href="https://github.com/TYPO3/Fluid/compare/2.3.4...2.4.0">https://github.com/TYPO3/Fluid/compare/2.3.4...2.4.0</a></p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3534702017-12-15T12:47:43ZAlexander Opitzopitz.alexander@googlemail.com
<ul></ul><p>ATM the discussion is on the point, that we will do a 2.3.5 release in a 2.3 branch, which includes the needed fixes from this issue ... as also fixes for other issues.</p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3544742018-01-08T19:03:26ZAlexander Opitzopitz.alexander@googlemail.com
<ul></ul><p>State: The fluid 2.3 branch was created and pull request with latest fixes was created: <a class="external" href="https://github.com/TYPO3/Fluid/pull/364">https://github.com/TYPO3/Fluid/pull/364</a></p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3565542018-02-01T14:10:11ZHelmut Hummeltypo3@helhum.io
<ul></ul><blockquote>
<p>For background information: What it would mean to update to 2.4:</p>
</blockquote>
<p>TYPO3 8.7 marked is compatible with fluid ^2.3<br /><a class="external" href="https://github.com/TYPO3/TYPO3.CMS/blob/f368dedac509b4d8284fc0353cda668ab3053df2/composer.json#L52">https://github.com/TYPO3/TYPO3.CMS/blob/f368dedac509b4d8284fc0353cda668ab3053df2/composer.json#L52</a></p>
<p>Users installing TYPO3 with composer will get fluid 2.4. If this is a problem we should take action soon (and maybe rethink the fluid release strategy)</p>
<blockquote>
<p>ATM the discussion is on the point, that we will do a 2.3.5 release in a 2.3 branch</p>
</blockquote>
<p>If TYPO3 8.7 is compatible with fluid 2.4, then branching fluid isn't necessary.<br />If TYPO3 8.7 isn't compatible, we need to change the composer requirement and disallow fluid 2.4, which would be breaking somehow.</p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3566802018-02-03T17:50:39ZSusanne Moogsusanne.moog@typo3.org
<ul><li><strong>Target version</strong> set to <i>8.7.10</i></li></ul> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3568352018-02-05T09:06:35ZAndreas Kienast
<ul><li><strong>Related to</strong> <i><a class="issue tracker-4 status-5 priority-4 priority-default closed" href="/issues/83774">Task #83774</a>: Update typo3fluid/fluid to 2.4.0</i> added</li></ul> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3568382018-02-05T09:07:52ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Under Review</i></li></ul><p>Patch set 2 for branch <strong>TYPO3_8-7</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/55555">https://review.typo3.org/55555</a></p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3568402018-02-05T09:08:55ZGerrit Code Review
<ul></ul><p>Patch set 3 for branch <strong>TYPO3_8-7</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/55555">https://review.typo3.org/55555</a></p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3568502018-02-05T10:52:51ZGerrit Code Review
<ul></ul><p>Patch set 1 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/55560">https://review.typo3.org/55560</a></p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3568532018-02-05T11:03:24ZGerrit Code Review
<ul></ul><p>Patch set 4 for branch <strong>TYPO3_8-7</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/55555">https://review.typo3.org/55555</a></p> TYPO3 Core - Bug #82227: count(): Parameter must be an array or an object that implements Countablehttp://forge.typo3.org/issues/82227?journal_id=3568582018-02-05T11:48:39ZChristian Kuhnlolli@schwarzbu.ch
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Closed</i></li></ul><p>resolved by <a class="issue tracker-4 status-5 priority-4 priority-default closed" title="Task: Update typo3fluid/fluid to 2.4.0 (Closed)" href="http://forge.typo3.org/issues/83774">#83774</a></p>