🇬🇧United Kingdom @a.hover

Account created on 4 December 2012, almost 12 years ago
#

Recent comments

🇬🇧United Kingdom a.hover

Rerolled the patch in #16 to the latest version of 2.0.x.

🇬🇧United Kingdom a.hover

I'm so sorry -t here was an issue with the previous patch as well.
This is what I get for rushing things! 3rd time's the charm, hopefully.

🇬🇧United Kingdom a.hover

Apologies, I forgot the argument in the service!

🇬🇧United Kingdom a.hover

Hi @wengerk,

I am re-rolling the patch above for the latest 6.0.x (6.0.2).
I have addressed the dependency injection, but not had chance to look at the KernelTest items.

Thanks,
Ali

🇬🇧United Kingdom a.hover

This patch appears to resolve this problem.
However, not sure if this is the best place for the code or if the original code should be changed from:
$ids = &drupal_static(__FUNCTION__ . '_ids', FALSE);
To
$ids = &drupal_static(__FUNCTION__ . '_ids', []);

Not wanting to disrupt things too much, so have opted for the patch version instead.

🇬🇧United Kingdom a.hover

@coaston sounds like this is a similar issue to https://www.drupal.org/project/admin_toolbar/issues/3385337 🐛 admin_toolbar_tools more than doubles number of queries needed during site-install degrading performance Active

🇬🇧United Kingdom a.hover

Re-rolling the patch in #8 to work with the latest version of this module (8.x-1.3).

🇬🇧United Kingdom a.hover

I can confirm the work in MR 51 has solved the issue for me where users were seeing other users' flagged content in a view with a Flag relationship.

🇬🇧United Kingdom a.hover

Hi Phil! Hope you're well. Thanks for adding Dejan as a maintainer!
@vdovhaliuk is also working at CTI - sounds like Dejan should have access to add him now anyway, so thanks!

🇬🇧United Kingdom a.hover

Hello,

Thanks for this great module - it saved me a bunch of custom work :)
Attached is a patch for including the custom message when exceeding the flag limit.

I've also updated the default message to make it more grammatically correct, I hope that's ok.

Thanks,
Ali

🇬🇧United Kingdom a.hover

Update the patch above to bring inline with 1.9 release and to prevent duplicate update hook.

🇬🇧United Kingdom a.hover

We've had the same issue. Attaching a patch for this.

🇬🇧United Kingdom a.hover

Attached is a patch created to resolve the issues we're seeing.
We still use the original $entity value for everything else since this seems to be correct in all other cases.
The identified $entity_latest revision is used for the status info.

Production build 0.71.5 2024