Account created on 28 April 2008, over 16 years ago
#

Recent comments

πŸ‡©πŸ‡ͺGermany zany

The "2.0.0-alpha1" is a tag on the "8.x-1.x" branch. You can download a zip (button "Code") here https://git.drupalcode.org/project/views_xml_backend
Or wait a few weeks, we are in the process of sorting this out.

πŸ‡©πŸ‡ͺGermany zany

I tried to contact some of the current maintainers by different channels, but couldn't get any response.

πŸ‡©πŸ‡ͺGermany zany

I contributed to needed compatibility patches that need to be integrated for proper D10 support.

I second this application for co-maintainership.

πŸ‡©πŸ‡ͺGermany zany

To get the whole MR as patch use the MR URL (highlighten green) and add .patch or .diff.
E.g. https://git.drupalcode.org/project/views_xml_backend/-/merge_requests/4....

πŸ‡©πŸ‡ͺGermany zany

There are now two readme files in the packaged zip:

unzip -l editor_advanced_link-2.2.5.zip |rg -i readme
     2269  08-07-2024 17:55   editor_advanced_link/README.md
     1942  08-07-2024 17:55   editor_advanced_link/readme.md

This will be a major problem for case-insensitve filesystems (Windows, macOS).

Composer may break with

Update of drupal/editor_advanced_link failed
 5/5 [============================] 100%
In ZipDownloader.php line 172:
  Failed to extract drupal/editor_advanced_link: (1) '/usr/bin/unzip' -qq 'vendor/composer/tmp-xyz.zip' -d 'vendor/composer/xyz'
  replace vendor/composer/xyz/editor_advanced_link/readme.md? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
  (EOF or read error, treating as "[N]one" ...)
πŸ‡©πŸ‡ͺGermany zany

There is already a MessengerTrait in QueryPluginBase and TestMessenger can be injected from ViewsXmlBackendTestBase.
The changes should look more like this:

πŸ‡©πŸ‡ͺGermany zany

Thanks for your offer to help out. Appreciated.

πŸ‡©πŸ‡ͺGermany zany

@joelseguin @protitude the patch #14 is recommended and proposed, it does not change the calling parameters.

πŸ‡©πŸ‡ͺGermany zany

Just a heads up: it can still be reproduced with 3.x.

Going from 1.2 to 3.x one of the file names should be
spamspan/src/TwigExtension/SpamspanExtension.php (1.3 and later)
but this file name might be stuck (depending on file system and used tools)
spamspan/src/TwigExtension/SpamSpanExtension.php (1.2)

Still see #9 for a solution if this affects you. There is nothing to fix/change in the project though.

πŸ‡©πŸ‡ͺGermany zany

Confirmed this fix works with D9.5.8

@wheelercreek the patch is the drupalcode link above, or as plain diff it's
https://git.drupalcode.org/issue/permissions_by_term-3354478/-/commit/4d...

E.g. in composer.json under extras patches insert:

"drupal/permissions_by_term": {
  "Fix Twig regression patch": "https://git.drupalcode.org/issue/permissions_by_term-3354478/-/commit/4daf422f379585da550a05d23d4c77aa8dc328ca.diff"
}
πŸ‡©πŸ‡ͺGermany zany

@crutch my patch (#14) is based on 3.1.21. If all else (composer?) fails just change that one line by hand (insert = 0 two times).

https://git.drupalcode.org/issue/permissions_by_term-3231556/-/commit/42...

πŸ‡©πŸ‡ͺGermany zany

As Chris--S said it's better to just add the defaults to the args after langcode.
I've created a branch with just that fix which should be uncontroversial to apply.

In the meantime apply a patch from https://git.drupalcode.org/issue/permissions_by_term-3231556/-/commit/42...

E.g. in composer.json under extras patches insert:

"drupal/permissions_by_term": {
  "Fix optional parameters patch": "https://git.drupalcode.org/issue/permissions_by_term-3231556/-/commit/421de6c96e79e25225a04aa2ab4043245b61fc01.diff"
}
Production build 0.71.5 2024