- 🇺🇸United States smustgrave
Have to be in agreement with @jungle in #19. Seems this patch was just covering a symptom but the cause has been fixed.
If anyone disagrees please reopen with an explanation why.
Thanks!
- heddn Nicaragua
I wonder if closing won't fix is quite what we want to do? Could we complain more loudly or do something to help track down these issues? I regularly see these warnings in test runs and its a bear to figure out where the null is coming from. Being more loud in our logging of the source would be really nice.
- Status changed to Needs review
over 1 year ago 8:28pm 15 May 2023 - last update
over 1 year ago 29,369 pass, 2 fail - 🇺🇸United States joegraduate Arizona, USA
We're seeing these warning show up in various contexts when using exported configuration entities provided by custom modules in
config/install
directories that do not have UUIDs. It looks like theDrupal\Core\Config\Entity\ConfigEntityBase::preSave()
method usesDrupal\Core\Entity\EntityBase::uuid()
which returnsNULL
if a UUID doesn't exist.The attached patch changes those NULL values into empty strings in
Drupal\Core\Config\Entity\ConfigEntityBase::preSave()
to prevent them from getting passed as query condition values. - last update
over 1 year ago 30,315 pass, 2 fail - Status changed to Needs work
over 1 year ago 8:40pm 15 May 2023 - 🇺🇸United States smustgrave
The issue summary should be updated with the new proposed solution.
Will also require a test case to show the problem.
- last update
over 1 year ago 29,376 pass, 2 fail - last update
over 1 year ago 28,499 pass, 2 fail - last update
over 1 year ago 29,376 pass, 2 fail - 🇨🇦Canada joseph.olstad
Got this notice in D10 using php 8.1 when switching text format from ckeditor5 to ckeditor4
patch 25 resolves the issue. - 🇷🇴Romania lucian.ilea
Hi!
I confirm that #13 fixed the issue for me on Drupal 9.5.9 with php 8.1.
Thank you! - heddn Nicaragua
Part of why this is becoming an issue, more and more is because of 🐛 'filter.format.' . WebformHtmlEditor::DEFAULT_FILTER_FORMAT missing UUID Fixed .
- First commit to issue fork.
- Merge request !4263Issue #3302838: Querying with NULL values results in warning mb_strtolower(): Passing null to parameter is deprecated → (Open) created by suryabhi
- Open on Drupal.org →Environment: PHP 8.2 & MySQL 8last update
about 1 year ago Not currently mergeable. - Status changed to Needs review
about 1 year ago 5:36am 27 June 2023 - last update
about 1 year ago 29,559 pass - Status changed to Needs work
about 1 year ago 12:27pm 27 June 2023 - last update
about 1 year ago 29,559 pass - last update
about 1 year ago 29,559 pass - last update
about 1 year ago 29,559 pass - last update
about 1 year ago Patch Failed to Apply - last update
about 1 year ago Patch Failed to Apply - last update
about 1 year ago Patch Failed to Apply - Status changed to Needs review
about 1 year ago 11:18am 5 July 2023 - last update
about 1 year ago 29,801 pass - 🇮🇳India mrinalini9 New Delhi
Rerolled patch #36, please review it.
Thanks!
- Status changed to Needs work
about 1 year ago 11:34am 5 July 2023 - 🇺🇸United States smustgrave
Thank you for the interest but please read the tags. Previously tagged for issue summary update and tests.
Also the MR is on the correct branch and should continue there.
- last update
about 1 year ago 29,797 pass - last update
about 1 year ago 29,802 pass - Status changed to Needs review
about 1 year ago 9:11am 24 July 2023 - Status changed to Needs work
about 1 year ago 10:02am 24 July 2023 - last update
about 1 year ago 29,957 pass, 1 fail - 🇫🇷France Bapt.
Merge request !4263 fixed my issue on Drupal 10.1.2 (PHP 8.1.14 / MySQL 5.7.29)
Thanks.
- last update
about 1 year ago 29,958 pass - last update
about 1 year ago 29,958 pass - last update
about 1 year ago 30,060 pass I've been unable to reproduce this by enabling modules. Attempted several contrib modules (including ckeditor 4 and 5) in fresh Drupal 10.1.x install.
- 🇨🇦Canada joseph.olstad
@bjc2265, are you using PHP 8.0 or PHP 8.1+ ?
This fix is for PHP 8.1+
Using PHP 8.1.8 and Drupal 10.1.4
Just to confirm: message should appear in GUI after going to Extend tab, selecting a module that changes configuration (such as Configuration Translation or Content Translation) and clicking Install? Because doing that on fresh install I'm not seeing error in logs or as alert.
- First commit to issue fork.
- Assigned to PrabuEla
- Issue was unassigned.
- Status changed to Needs review
7 months ago 7:59pm 12 February 2024 - 🇳🇴Norway eiriksm Norway
I don't know how to do test only patches with gitlab. I also was not able to "claim" push access to the fork
Anyway. Here is a test only patch, please feel free to add to the fork
- Status changed to Needs work
7 months ago 8:09pm 12 February 2024 - 🇺🇸United States smustgrave
If you click "Get push access" there's an option on the pipeline https://git.drupalcode.org/issue/drupal-3302838/-/pipelines/56031
There you can click "Test-only feature"
Current MR appears to have test failures.
Was previously tagged for issue summary too as some sections appear to be missing.
- 🇳🇴Norway eiriksm Norway
I did click the "get push access" and have done so many times before as well. But some hiccup today probably? Tried again now, and now it works :)
Let me try to rebase and get the test only thing in there as well, never tried that before.
Leaving on NW for the issue summary part
- 🇺🇸United States smustgrave
Now will say there's a bug/feature being worked out where if the MR is opened by a core committer, even getting push access you can't run the test-only feature or rerun any tests.
- 🇳🇴Norway eiriksm Norway
right. was probably just a hiccup I guess, since both work now.
Link to test-only run: https://git.drupalcode.org/issue/drupal-3302838/-/jobs/799034
- Status changed to RTBC
about 1 month ago 9:14pm 28 July 2024 - Status changed to Needs work
about 1 month ago 8:16am 29 July 2024