We were also encountering this issue when granting permissions via update hooks.
As a workaround, we added an update hook to manually revoke the permissions:
$storage = \Drupal::entityTypeManager()->getStorage('taxonomy_vocabulary');
$vocabularies = [
'article_categories',
'codes',
'event_categories',
];
foreach ($vocabularies as $vid) {
$vocabulary = Vocabulary::load($vid);
if ($vocabulary) {
$permissions = [
"create terms in $vid",
"delete terms in $vid",
"edit terms in $vid",
];
user_role_revoke_permissions('superuser', $permissions);
$storage->delete([$vid => $vocabulary]);
}
We haven't tested the original issue since then.
Here's the commit in the 6.0.x branch - this seems to be included in release 6.0.6, although the commit message doesn't mention this issue.
Duplicate of 🐛 Metrics command generates error Fixed .
We should release a new version with this fix.
malcomio → created an issue.
malcomio → created an issue.
Updated to include fetching thumbnail, as discussed on 📌 Implement a getName method in the VideoEmbedProvider Needs review
See also 📌 Implement a getName method in the VideoEmbedProvider Needs review - it would make sense to also implement a getName method.
I'd tried to apply these changes on top of my changes there, but they conflict.
Will try to combine them when I can.
Having updated to the latest version of wse, we have been able to uninstall as expected.
malcomio → created an issue.
As mentioned in #11, the current merge request includes various other changes that don't relate to this issue.
It makes sense to keep changes limited to the issue in question, and fix coding standards issues separately.
https://git.drupalcode.org/project/views_natural_sort/-/merge_requests/13 just addresses the deprecations in src/ViewsNaturalSortService.php
malcomio → made their first commit to this issue’s fork.
I think that the relevant change is the addition of the route subscriber in https://git.drupalcode.org/project/wse/-/commit/69838e13489891a9995fe8f8...
malcomio → created an issue.
Thanks - perhaps the issue has already been fixed.
We have a commit of wse that is 2 months old, and Drupal core is on version 10.2.5.
Will test again with updates.
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
This limitation of workspaces isn't obvious from the documentation - I've added some info to https://www.drupal.org/docs/8/core/modules/workspace/using-workspaces →
malcomio → created an issue.
I've also encountered this issue when installing 4.2.6 via an update hook.
This seems odd, because a similar pattern can be seen in getSubscribedEvents
in lots of other modules, e.g.
https://git.drupalcode.org/project/devel/-/blob/5.x/src/EventSubscriber/...
https://git.drupalcode.org/project/feeds/-/blob/8.x-3.x/src/EventSubscri...
I realised that this belongs in drush, not core: https://github.com/drush-ops/drush/issues/5933
malcomio → created an issue.
I think that this is an instance of https://github.com/phayes/geoPHP/issues/181
Given that the composer.json for views_geojson refers to https://github.com/itamair/geoPHP, it's odd that we're getting this error, although I can see that our project declares a dependency on the phayes version directly for some reason, so maybe that's why we're seeing it.
Even if that's the reason we're seeing it, it would make sense to catch any exceptions.
malcomio → created an issue.
malcomio → created an issue.
Thanks
Thanks - the link in the issue summary responds with a 404, which suggests https://learn.microsoft.com/en-us/bing/search-apis/bing-custom-search/re...
Superseded by 📌 Automated Drupal 11 compatibility fixes for ckeditor_datetime Needs review
The patch applies cleanly, but there is a warning: 4 lines add whitespace errors.
It is a dependency for the dp_faq submodule, so we'd need to think about how best to address that.
malcomio → made their first commit to this issue’s fork.
Thanks, but the 1.x branch is no longer supported, and the patch no longer applies.
malcomio → made their first commit to this issue’s fork.
malcomio → created an issue.
malcomio → created an issue.
The patch on #7 is working as expected for me.
+1 for RTBC
Thanks - I hadn't realised that the field I was trying to display was actually a Geolocation Geometry field.
Not sure if it actually makes sense to provide a latitude/longitude formatter for that type of field, given that they're unlikely to have a single location in.
malcomio → created an issue.
See also #2842386: [Regression] Restore Javascript APi → and ✨ Allow AjaxResponse to be altered RTBC
Seems like this can already be achieved using a flag.html.twig template, or template_preprocess_flag.
Maybe worth considering as a default in the module?
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
One option might be to have an entity type for the links, and flag those entities.
We'd need to have a mechanism to create those entities from an API request or form submission.
malcomio → created an issue.
LGTM
malcomio → created an issue.
In the project I'm working on, a lot of the information actually comes from search_api_solr → - it may be necessary to do something in that module as well as or instead of the main search_api module.
After disabling wse_config, I encountered a different error:
php-error web-46717 [16-Feb-2024 14:57:08 Europe/London] PHP Fatal error: Aborting! The New Relic imposed maximum PHP function nesting level of '5000' has been reached. This limit is to prevent the PHP execution from catastrophically running out of C-stack frames. If you think this limit is too small, adjust the value of the setting newrelic.special.max_nesting_level in the newrelic.ini file, and restart php. Please file a ticket at https://support.newrelic.com if you need further assistance. in /mnt/www/html/docroot/core/lib/Drupal/Core/Session/AccountProxy.php on line 91 request_id="v-a79fef58-ccdb-11ee-b56c-27a30e76dfdd"
malcomio → created an issue.
malcomio → created an issue.
Thanks - I tried the same with the dev version, and the indexing completed without problems.
Will also try the patch with 8.x-1.31
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
malcomio → created an issue.
malcomio → made their first commit to this issue’s fork.
Thanks - impressively quick to create an issue on a module that was only created a few hours ago :)
This issue should already be fixed by https://git.drupalcode.org/project/unaggregated/-/commit/dffbaf8ea569caf...
malcomio → created an issue.
As noted on 🐛 Incompatible with devel: "Some mandatory parameters are missing ("header") to generate a URL for route "entity.response_header.devel_load"" Active the patch seems to fix the issue.
This issue should be re-opened - the merge request is still open, and https://git.drupalcode.org/project/http_response_headers/-/blob/8.x-2.x/... still shows the old code.
I think that this is a duplicate of 🐛 Error 500 - MissingMandatoryParametersException Fixed , although because that one is closed, I'd suggest we leave this open until a maintainer addresses it.
The patch on that issue does seem to fix the problem.
Thanks - workspace preview seems to do exactly what I was asking for here.
Yes - it does seem to be the same issue, and the equivalent fix seems to address it.
I don't think that this is specific to 11.x after the changes for 🐛 Restrict access to empty top level administration pages Fixed - see 🐛 Users with access taxonomy overview permission cannot access admin/structure Needs review , which is on 10.2.x
Merge request 6234 is based on MR 6100 against 11.x for 🐛 Admin page access denied even when access is given to child items RTBC , but doesn't include the tests.
It seems to work as expected from some very limited testing so far.
malcomio → created an issue.