- Issue created by @ryanbuckley@gmail.com
- First commit to issue fork.
- Merge request !6482Update ClaroPreRender.php, confirming classes provided are in array format → (Open) created by sijumpk
- Status changed to Needs review
about 1 year ago 7:25am 7 February 2024 - 🇮🇳India sijumpk
'
class
' attribute should be an array. Seems like in case of Geofield Map, its passing a string instead of an array. Because of that this error is happening. Noticed that inside core code, there are some places where double checking for array elements are happening. Eg inside ClaroPreRender::verticalTabs$groups_are_present = isset($element['group']['#groups']) && is_array($element['group']['#groups']);
So adding the same type of checking for $
element['#attributes']['class']
also. - Status changed to Needs work
about 1 year ago 3:31pm 8 February 2024 - 🇺🇦Ukraine dburiak
Here is the test-only patch showing the issue.
Without patching, the test will fail. - last update
about 1 year ago Patch Failed to Apply - Status changed to Needs review
about 1 year ago 7:42pm 11 February 2024 - Status changed to Needs work
about 1 year ago 3:11pm 12 February 2024 - 🇺🇸United States smustgrave
Left a small comment but can you update MR for 11.x please
- First commit to issue fork.
- 🇺🇦Ukraine anpolimus Kyiv
@smustgrave, do you mean creating a new MR against 11.x or updating this one from 10.x to 11.x?
- Merge request !6560Update ClaroPreRender.php, confirming classes provided are in array format → (Closed) created by dburiak
- 🇺🇦Ukraine dburiak
Added MR to 11.x: https://git.drupalcode.org/issue/drupal-3419621/-/tree/3419621-typeerror...
Please review. - Status changed to Needs review
about 1 year ago 4:26pm 20 February 2024 - Status changed to Needs work
about 1 year ago 8:15pm 20 February 2024 - 🇺🇸United States smustgrave
Issue summary appears to be missing proposed solution section.
Also wonder if there's been a backtrace to see what's calling this with a string. In case we covering up a larger issue.
- 🇺🇸United States lpeabody
As an FYI this error is resolved in the case of the Geofield Map module in version 3.0.15 https://git.drupalcode.org/project/geofield_map/-/compare/3.0.14...3.0.1...
- 🇺🇦Ukraine dburiak
@smustgrave The string in the original issue was calling from Geofield Map module (already fixed). The summary is updated with a proposed solution section.
- Status changed to Needs review
11 months ago 5:28pm 26 March 2024 - Status changed to Needs work
11 months ago 9:56pm 30 March 2024 - 🇺🇸United States smustgrave
But what are the steps to reproduce in core without the geofield module?
There are 2 MRs up, can one please be hidden.
- Status changed to Needs review
11 months ago 2:26pm 3 April 2024 - Status changed to RTBC
11 months ago 3:03pm 4 April 2024 - 🇺🇸United States smustgrave
Thanks @pameeela for updating that.
Test only feature shows the issue
1) Drupal\KernelTests\Core\Theme\ClaroTableTest::testThemeTablePositionStickyPreRender PHP Error was thrown. /builds/issue/drupal-3419621/core/tests/Drupal/KernelTests/Core/Theme/ClaroTableTest.php:74 /builds/issue/drupal-3419621/vendor/phpunit/phpunit/src/Framework/TestResult.php:728 FAILURES! Tests: 2, Assertions: 3, Failures: 1.
Additional check seems to solve the issue.
- Status changed to Needs work
11 months ago 10:01am 5 April 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
I left some small nits on the MR to address. Once they are this can go back to rtbc.
- Status changed to Needs review
11 months ago 5:30pm 5 April 2024 - Status changed to RTBC
11 months ago 4:54am 6 April 2024 - Status changed to Needs work
11 months ago 12:33pm 6 April 2024 - 🇫🇷France nod_ Lille
Still need to remove the try/catch. Tests are fast now, it's not a problem if it stops everything else after.
- Status changed to Needs review
11 months ago 12:11pm 7 April 2024 - 🇦🇺Australia pameeela
Accepted the suggestion to remove the try/catch, the tests are failing but I can't see that it's related to this?
- Status changed to RTBC
11 months ago 2:54pm 7 April 2024 - Status changed to Fixed
11 months ago 8:52am 8 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.