Brisbane!
Account created on 5 November 2014, over 10 years ago
#

Merge Requests

More

Recent comments

🇦🇺Australia jannakha Brisbane!

thanks for you contribution!
Tested MR16 - looks good.

Screenshot before patch:

Screenshot after patch:

🇦🇺Australia jannakha Brisbane!

So my test case is:
I'm moving access checks from hook_ENTITY_TYPE_access into RouteSubscriber.

I've got a custom RouteSubscriber which adds Requirement with '_custom_access' based on user/entity on routes:
$collection->get('entity.node.delete_form')
$collection->get('entity.node.edit_form')

the routes are not accessible when conditions are met - as expected
but on views Edit/Delete operations are visible (although they lead to access denied/error page)

after applying patch #135 (to Drupal 10.4.5) - views operations are displayed as expected (no edit/delete ops when they should not be) just like when access was handled in hook_ENTITY_TYPE_access.
hurray!

what else is required to release this issue?

PS now the issue is "Delete" op appears on "Edit" form when it's forbidden by RouteSubscriber/custom access execution.
If I won't find an existing issue, I'll create a new issue for that. here's existing issue for Media https://www.drupal.org/project/drupal/issues/2998824 🐛 MediaAccessControlHandler update/delete access caching is not correct Needs work

PPS back to hook_ENTITY_TYPE_access for now (unless there are better suggestions?)

🇦🇺Australia jannakha Brisbane!

Thanks for your contribution!

the failed MR status points to a failed unit test - please fix the test.

🇦🇺Australia jannakha Brisbane!

@ooa33 - do you mind putting your code as a merge request?

🇦🇺Australia jannakha Brisbane!

Isn't for accessibility colour/visuals should not be the only difference in the meaning of the links?

GitHub has tags for different releases:
https://github.com/vercel/next.js/releases

🇦🇺Australia jannakha Brisbane!

Isn't for accessibility colour should not be the only difference in the meaning of the links?

GitHub has tags for different releases:
https://github.com/vercel/next.js/releases

🇦🇺Australia jannakha Brisbane!

+1. Tested and ready to be released.

🇦🇺Australia jannakha Brisbane!

@lendude exactly! #10 is just treatment of a symptom, not an actual solution to the potentially bigger issue.

If EntityField::buildGroupByForm() is not called, then EntityField::submitGroupByForm() should not be called.
EntityField::submitGroupByForm() expects certain fields to be there (hence the null exception).

I haven't found a reason why EntityField::buildGroupByForm() is not called but EntityField::submitGroupByForm() is called on aggregations like min/max/sum/etc, but when it's default (eg group) both EntityField::buildGroupByForm() and EntityField::submitGroupByForm() are called.

🇦🇺Australia jannakha Brisbane!

jannakha changed the visibility of the branch 3513598-typeerror-array_filter to hidden.

🇦🇺Australia jannakha Brisbane!

jannakha changed the visibility of the branch 3513598-typeerror-arrayfilter-argument to hidden.

🇦🇺Australia jannakha Brisbane!

Further investigation of the issue:
- when Use aggregation is on, fields on the view have "Aggregation settings"
- if Aggregation type set to "Group results together" other form options (Group column and Group columns (additional)) are rendered
- if Aggregation type set to Max/Min/etc other form options are not rendered, but they are being processed by submitGroupByForm() in web/core/modules/views/src/Plugin/views/field/EntityField.php
- if Aggregation type set to other than "group" - EntityField::submitGroupByForm() still processes although EntityField::buildGroupByForm() was never called (only HandleBase::buildGroupByForm() is called)

see attached screenshots

🇦🇺Australia jannakha Brisbane!

jannakha changed the visibility of the branch 2909372-enable-drupal.commenting.variablecomment.missingvar-coding to hidden.

🇦🇺Australia jannakha Brisbane!

jannakha changed the visibility of the branch 2909372-enable-drupal.commenting.variablecomment.missingvar-coding to active.

🇦🇺Australia jannakha Brisbane!

Patch works as expected and fixes the issue.
+1 for RTBC

🇦🇺Australia jannakha Brisbane!

I found the order of menu items goes crazy if I update order of menu items in menu management interface: admin/structure/menu/manage/main (if it's your main menu)

If I update the actual structure of taxonomies - then menu renders correctly.

Check your weight values in config: config/sync/default/core.menu.static_menu_link_overrides.yml

I wonder if there should be feature to reset menu items weights based on taxonomy weights.

🇦🇺Australia jannakha Brisbane!

I tried to reproduce it on 10.4 with taxonomy_menu version v3.7.0 and it works without any errors.
Please update status to Closed (outdated)

🇦🇺Australia jannakha Brisbane!

works now on php 8.3
thanks for the patch

🇦🇺Australia jannakha Brisbane!

@veronicaseveryn thanks for patch!
+1 for patch #2

changing priority to critical

🇦🇺Australia jannakha Brisbane!

Can you please provide more details?
I've tested core.base_field_override.node.mdb_page.promote.yml on 10.4 - I didn't find any issues with export/import.

Here's example of config:

uuid: e1ff4807-ef12-49c0-ac95-2c1100875e5c
langcode: en
status: true
dependencies:
  config:
    - node.type.mdb_page
id: node.mdb_page.promote
field_name: promote
entity_type: node
bundle: mdb_page
label: 'Promoted to front page ok'
description: 'generic description of a field'
required: false
translatable: true
default_value:
  -
    value: 1
default_value_callback: ''
settings:
  on_label: 'On'
  off_label: 'Off'
field_type: boolean
🇦🇺Australia jannakha Brisbane!

I've tested MR - it looks like it's working with media as required.

I think it still needs more testing - especially for security/permissions.

@torfj - what were your test cases for term permissions?

🇦🇺Australia jannakha Brisbane!

all good - ready to go Drupal 11!

thanks for your contributions!

🇦🇺Australia jannakha Brisbane!

Both MR70 and #20 patches work and fix the issue.

MR70 will require developers to update any custom twig templates.

🇦🇺Australia jannakha Brisbane!

is it possible to create a merge request out of the patch for review?

🇦🇺Australia jannakha Brisbane!

1.4.10 is exporting these fields recursively

🇦🇺Australia jannakha Brisbane!

I found why this issue is happening! (unfortunately, CKEditor is editable by users and they can copy paste things as they please)!

Steps to reproduce:

1. On a content node (article) insert into a text field's CKEditor source (a link to non-existent node):

<a href="/node/21991" target="_blank" data-entity-type="node" data-entity-uuid="30d77877-0000-0000-0000-14589bc25583" data-entity-substitution="canonical"><em><strong>HERE</strong></em></a>

2. Save node
3. Go to Export tab of this node
4. It will crash with same exception
TypeError: Drupal\single_content_sync\ContentExporter::isReferenceCached():

Proposed solutions:
- ignore invalid links, if $linked_entity is NULL don't try to add to $embed_entities
- report an error - link doesn't exist (so users will fix it and then do export)

Users are silly - they'll put all sorts of stuff into text fields, so

🇦🇺Australia jannakha Brisbane!

I can confirm this issue exists.
I've got the same issue on 1.4.10 (upgraded from 1.4.8 - which worked fine)
doesn't happen on all content types, I'm investigating which field causing the issue

🇦🇺Australia jannakha Brisbane!

+1 as well. Tested on Drupal 10.4.1 and it works.

🇦🇺Australia jannakha Brisbane!

+1 RTBC
thanks for your contribution!

Please release :-)

🇦🇺Australia jannakha Brisbane!

phpunit is green. Good to go.
Manually tested on D11.
+1 for RTBC

🇦🇺Australia jannakha Brisbane!

Test on Drupal 11.
phpstan passes.

🇦🇺Australia jannakha Brisbane!

jannakha created an issue.

🇦🇺Australia jannakha Brisbane!

Looks good. phpstan is greem and works as expected.
Should create separate issue for CI and phpunit.

🇦🇺Australia jannakha Brisbane!

Thank you! Committed! 🍻

🇦🇺Australia jannakha Brisbane!

Looks good as works as expected. Thank you.

🇦🇺Australia jannakha Brisbane!

+1. Would be good to have security coverage.

🇦🇺Australia jannakha Brisbane!

Looks great and works as expected.

🇦🇺Australia jannakha Brisbane!
🇦🇺Australia jannakha Brisbane!
🇦🇺Australia jannakha Brisbane!

First settings test added.

🇦🇺Australia jannakha Brisbane!

jannakha made their first commit to this issue’s fork.

🇦🇺Australia jannakha Brisbane!

jannakha made their first commit to this issue’s fork.

🇦🇺Australia jannakha Brisbane!

jannakha made their first commit to this issue’s fork.

🇦🇺Australia jannakha Brisbane!

works as required.
Thank you for your contributions and support!
Hurray for D11

🇦🇺Australia jannakha Brisbane!

Tested on Drupal 11. File scanning is working with executable and not allowing to upload the file when execution is disabled.

🇦🇺Australia jannakha Brisbane!

@smustgrave I think there is some confusion.
D11 compatibility is not fully implemented. File scanner is not working.
Please do phpcs separately once 2.1.0 release is done.

🇦🇺Australia jannakha Brisbane!

jannakha made their first commit to this issue’s fork.

🇦🇺Australia jannakha Brisbane!

hook_file_validate() will be replaced with event handler in this task

issue #3423615 was not implemented for this module

🇦🇺Australia jannakha Brisbane!

tested and works as required

thanks for your contributions!

🇦🇺Australia jannakha Brisbane!

tested and works as required

🇦🇺Australia jannakha Brisbane!

works as required
thanks for your contribution

🇦🇺Australia jannakha Brisbane!

works as required
thanks for your contribution

🇦🇺Australia jannakha Brisbane!

works as required.
thanks for your contribution

🇦🇺Australia jannakha Brisbane!

works as required.
thanks for your contribution.

🇦🇺Australia jannakha Brisbane!

works as required.
thanks for your contribution.

🇦🇺Australia jannakha Brisbane!

works as required.
thanks for your contribution.

🇦🇺Australia jannakha Brisbane!

Changes look good.
Tested.
Ready for release.

Thanks for your contribution!

🇦🇺Australia jannakha Brisbane!

- error goes away after "Term Merge" module is enabled
- cleaned up taxonomy manager merge library

🇦🇺Australia jannakha Brisbane!

Please don't put multiple issues into a same ticket - it makes it hard to test and review:
- new permissions structure
- add pagination on 'admin/structure/taxonomy_manager/voc' page

When status of ticket goes into "needs review" please provide screenshots and pages to be reviewed

🇦🇺Australia jannakha Brisbane!

see comments on MR

basically:
config has Editor has create permissions:

Running debug shows different values for different ways of testing permissions:

$taxonomy_vocabulary->access('create') is buggy
see https://www.drupal.org/project/drupal/issues/2886800 🐛 EntityAccessControlHandler::createAccess() and EntityAccessControlHandler::access() return false positive cache hits because it ignores context Needs work

🇦🇺Australia jannakha Brisbane!

- added extra library to JS from CDN
- requires https://www.drupal.org/project/taxonomy_manager/issues/3046752 🐛 AJAX error when editing term content Postponed: needs info patch #31 (MR49) to work correctly (which includes https://www.drupal.org/project/taxonomy_manager/issues/3474919 🐛 Form element taxonomy_manager_tree broken Active )

🇦🇺Australia jannakha Brisbane!

https://www.drupal.org/project/taxonomy_manager/issues/3046752 🐛 AJAX error when editing term content Postponed: needs info is required for any ajax requests to work

🇦🇺Australia jannakha Brisbane!

created a patch from MR (it's merged with #3474919)
make sure library/jquery.fancytree contains all JS - see #3467549 Change require custom libraries to suggestions as a better method with more options Needs work

🇦🇺Australia jannakha Brisbane!

jannakha made their first commit to this issue’s fork.

🇦🇺Australia jannakha Brisbane!

tested MR2 - issue is fixed.

thanks for your work!

🇦🇺Australia jannakha Brisbane!

check your config has been updated by re-running block_class_update_20017()
this update transformed config from string to json and if your config is old block_class will keep crashing

🇦🇺Australia jannakha Brisbane!

+1 for patch #13 works on 10.3.10

thanks for the patch!

🇦🇺Australia jannakha Brisbane!

can someone review?
it'll be nice to get a release with this feature

🇦🇺Australia jannakha Brisbane!

just for future, let's not scope creep issues: add to wish list is totally independent from add to cart AJAX and could've been its own MR

@anas_maw - can you expand on your comments on your patch? what should be done with it?
@luksak can you please add documentation about "Add to wish list" to readme.md?

🇦🇺Australia jannakha Brisbane!

MR8 is ready for review

Production build 0.71.5 2024