Nice to see the CSRF token in place
What is the use for the destination query parameter in content_lock_js_url? This URL is only called in the background and not something a user actually visits?
I expected a CSRF token instead of a custom hash. Can you elaborate why the custom hash is preferred in this case?
The phpstan pipeline job findings are fixed in the merge request
The merge request implements compatibility with both Commerce 2.x and 3.x
This issue was reported earlier in 🐛 RuntimeException: Adding non-existent permissions to a role is not allowed. Needs review . See the related issue for a patch and possible workarounds.
I should have clarified the issue occurs when using the 'Lock form using JS' option.
I moved the test coverage to \Drupal\Tests\content_lock\FunctionalJavascript\ContentLockNodeTest::testContentLockWithTranslations with this option enabled so the issue is triggered correctly.
The stylelint pipeline job findings are fixed in the merge request
This was reported (and fixed) earlier in 📌 Nullable types must be explicit Active
I'll close this issue as a duplicate
This issue is being fixed in 📌 Fix phpstan pipeline job findings Active with a class alias
The phpstan pipeline job findings are fixed in the merge request
Claro is marked as internal and is allowed to break BC since https://www.drupal.org/node/2582945 → per https://git.drupalcode.org/project/drupal/-/blob/11.x/core/themes/claro/...
The merge request fixes the phpcs pipeline job findings
The merge request updates Claro's jQuery autocomplete enhancements to use (only) css
Works, thanks!
The merge request
- Removes outdated warning: Relying on a CDN for external libraries can cause unexpected issues with Ajax and BigPipe support
- Lowers the severity from error to warning as it was before 📌 Change the Webform libraries CDN warning to an error in the Status report (/admin/reports/status). Fixed
idebr → created an issue.
The merge request fixes the phpcs pipeline job findings
Drupal 11 compatibility was fixed in 📌 Drupal 11 compatibility fixes Active
Duplicate of 📌 Update CKEditor 5 to 45.0.0 Active ?
The merge request converts ExtraField plugin discovery to attributes
The merge request fixes the phpcs pipeline job
The merge request updates the project to consistently use LF line endings
This was fixed in 📌 Commerce 3.0 campatibility Active
A follow-up is available for the change in line endings, see 📌 Use LF line endings consistently Active
#12 Return a boolean is now still valid, but triggers a deprecation
MR 233 is patch #34
Updated merge request after 🐛 Users are able to block themselves from Drupal Active was committed
The merge requests fixes the phpunit tests
In the merge request, the module claims it is compatible with Drupal Commerce 3. This means the changes from 📌 Commerce 3.0 campatibility Active are required for compatibility.
In addition the MollieApiMock determines the payment status from the paid amount using incorrect comparisons, eg. 29.99 === 29.9900000. I updated MollieApiMock to compare Prices instead.
The functional tests are now green.
My project requirement is pretty basic:
- Upload a file in a node
- Automatically translate the file when translating the node using Deepl
This is supported by tmgmt, see ✨ Translate files / attachments Fixed
The priority for automatic translations for documents in my project has been lowered, so I don't expect to do any development on this feature in the short term
\Drupal:state() is now replaced with \Drupal::keyvalue('user_form_test') in the test coverage
The merge request fixes the remaining phpstan issue
The patch in #2 is now available as a merge request
The merge request does not lock the entity when adding a content translation
idebr → created an issue.
The merge request removes the obsolete drupalci.yml file from the project.
Fixed the last test failure.
A non-translatable string should just be 'string', see for reference: https://www.drupal.org/docs/drupal-apis/configuration-api/configuration-... →
This is being fixed in 🐛 Undefined function field_ui_entity_operation Active . I'll close this issue as a duplicate, so we can focus our efforts in the related issue.
This is being fixed in 🐛 Undefined function field_ui_entity_operation Active . I'll close this issue as a duplicate, so we can focus our efforts in the related issue.
The item category contains additional logic to add a URL attribute when it uses a term reference field.
Using a field that is not a term reference will now no longer trigger a PHP exception
Can now be 10.3 as 10.2 is end-of-life
📌 Test with drupal/video_embed_field 3.x for Drupal 11 compatibility Active has been committed.
Committed to allow for further testing.
idebr → created an issue. See original summary → .
The merge request fixes the phpcs pipeline job
I filed an issue to fix 1.x as well while it is still supported, see 🐛 [1.x] Fix phpstan and commerce plugin contructors Active
.ck-editor includes the toolbar. Let's use .ck-content so the css only applies to the editable content.
The issue summary is updated with the issue summary template heading.
Can you check the Grant type 'Refresh Token' is enable on your Consumer edit form?
The merge request adds 'roles' to AccountForm getEditedFieldNames() and flagViolations(), so Constraint violations are properly displayed as form errors.
Confirmed the issue still occurs in 11.x
Yes, we should be explicit about ^9.0 of league/oauth2-server due to PHP compatibility of method signatures