Updated the issue and created a PR pointing to the latest 3.x version.
You are right, even though it is not an explicit PHP 8.4 warning I thought it would be a good contribution.
I have removed the variable typing and defined in @docs that this is a mixed type.
The 3.x branch is still used by many users because of its compatibility with other technologies, I have corrected the warnings in this one.
For the 4.x branch is already solved, even though some documentation was missing regarding the variable that can be nullable so I have updated it.
Fixed \Drupal\paragraphs_browser\Form\ParagraphsBrowserForm explicit nullable types.
Added explicit nullable types to @docs.
Updated PR branch from origin/8.x-2.x
Fixed bug that caused fatal error from 3.0.0-alpha2
reroll
🐛
Previewing appears to unlock node
RTBC
on locked entity deletion.
Thanks for your contribution! I find it a good contribution to the module, I am merging your request in the main branch.
Rerolled patch #211 ✨ Taxonomy Index for unpublished entities Needs work for Drupal 11.1.1
Rerolled patch
#1
🐛
Database exception: Unknown column 'base.' when using entity types without bundle
Active
for branch 2.1.x.
I had a similar error:
[error] TypeError: Drupal\Component\Utility\Html::escape(): Argument #1 ($text) must be of type string, null given, called in /var/www/html/xyz/docroot/core/lib/Drupal/Component/Render/FormattableMarkup.php on line 256 in Drupal\Component\Utility\Html::escape() (line 431 of /var/www/html/xyz/docroot/core/lib/Drupal/Component/Utility/Html.php)
That I solved with this patch, if it helps anyone.
Rerrolled for 3.0.0-alpha2
Added missing core requirement to legacy module for 3.0.x rerroll.
Rerrolled for 3.0.x
Use Drupal #attributes to set wrapper attributes.
Updated the property key so that it does not conflict with the core rendering array system and added CSS classes to the template elements to improve theming.
Applied proposed resolution in a patch.
enchufe → changed the visibility of the branch 3469440-plugin-doesnt-work to hidden.
Proposed resolution's patch
Patch #2 💬 Multilingual support? RTBC works as expected
You are right
#5
🐛
Refine the process value filtering so that it gets values from the row
Postponed: needs info
, it is more convenient to check it with array_key_exists(), I update the PR and the patch.
enchufe → changed the visibility of the branch 3466235-refine-value-filtering to hidden.
Created a patch with the proposed resolution.
enchufe → created an issue.
Proposed resolution implemented.
enchufe → created an issue.
Moved update hook from save_edit.module to save_edit.install file to have the functions sorted out.
Proposed solution applied in a patch.
enchufe → created an issue.
After updating Drupal core to version 10.3 these hooks stopped working, as a solution I implemented the patch of this issue 💬 Create alter hook hook_paragraphs_widget_alter() Fixed and updated the hooks.
I recently updated the Drupal core version to 10.3 (From 10.1) and hook_field_widget_single_element_WIDGET_TYPE_form_alter() stopped working, it simply ignored the existence of the functions invoked with this hook. As a solution I applied patch
#3
💬
Create alter hook hook_paragraphs_widget_alter()
Fixed
and replaced the hooks and now it works correctly again.
This is an opportune time to evaluate retiring this module. The team at CKSource (developers of CKEditor) have released CKEditor 5 Plugin Pack → which includes the plugin provided by this module.
Thank you #3 ✨ Compatibility with field group Needs work ! Your documentation worked for me out of the box.
A colleague solved it with the
Migrate Files (extended) →
module.
Should the issue be closed or is it still a bug in this module?
Adds a validation of the existence of the variable on the line that indicates the warning.
@florianmuellerCH To manage Cookies I use the JavaScript library orestbida/cookieconsent. The GA4 beacon is loaded with the attributes type=text/plain data-category=analytics. And the rest is configured by the library by activating the script (removing type=text/plain) when consenting to Cookies.
This gave me quite a headache, I hope it helps!
+1 It is not specified on the main page of the module that it requires a paid subscription. It looks more like a proprietary promotion in open source.
#6
🐛
Error: Call to a member function getType() on bool in filefield_paths_filefield_paths_process_file()
Needs work
Currently I have the migration development paused, when I resume it I will test the patch.
#8
🐛
Error: Call to a member function getType() on bool in filefield_paths_filefield_paths_process_file()
Needs work
But, if $temporary_scheme_name is always NULL, wouldn't the "if" logic always be skipped?
If attributes are added in TWIG as raw, they are not displayed in the final HTML, and the async attribute is displayed as async="true".
Source: drupal/ga4_google_analytics:dev-1.1.x
Removed the "required" property from the new field in the configuration form.
enchufe → created an issue.
enchufe → created an issue.
enchufe → made their first commit to this issue’s fork.
None of these fixes are related to the
Tome Add Paths →
module, but to its parent
Tome →
module.
Please report this issue to the
Tome Drupal.org issue queue →
.
Rerolled patch #9 for version 2.x, passes the Google Lighthouse test.
As an alternative, I have created the submodule Tome Add Paths → .
Added to the event subscribers a specific one for the sitemap.
Added the limit in the form with the machine name bundle max lenght limit core constant. Also added a security check that programmatically limits machine name in case it exceeds the maximum allowed length.
The patch unnecessarily limits the machine name of the entity bundle, causing it to have a lower limit than normal.