Georgia (US)
Account created on 7 April 2005, about 20 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States agentrickard Georgia (US)

Right. For the first time in 18 (!) years, I am not using this module on client work, so I don't have time to maintain it.

@mably has been added with vcs perms and the permission to create releases.

@danrod has been added with vcs perms

🇺🇸United States agentrickard Georgia (US)

I have added you as a maintainer with commit, issue management, and release priveleges.

🇺🇸United States agentrickard Georgia (US)

Oddly, I can no longer reproduce this on Drupal 11.1.3

🇺🇸United States agentrickard Georgia (US)

agentrickard created an issue.

🇺🇸United States agentrickard Georgia (US)

This has been fixed now.

🇺🇸United States agentrickard Georgia (US)

Merged and left open for the bot.

🇺🇸United States agentrickard Georgia (US)

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

🇺🇸United States agentrickard Georgia (US)

Well, I just noticed that we are getting test failures on the bot patch. So those need to be looked at.

🇺🇸United States agentrickard Georgia (US)

I will go ahead and roll and official 2.0.2 and then we can see what else needs to be done.

Thank you!

🇺🇸United States agentrickard Georgia (US)

All that aside, this seems to be working as expected.

🇺🇸United States agentrickard Georgia (US)

It looks like 📌 Automated Drupal 11 compatibility fixes for cacheexclude Needs review only requires a change to the info file.

If you can test that issue and confi9rm that it works on Drupal 11, I can roll a point release.

🇺🇸United States agentrickard Georgia (US)

I have made some comments on the new MR. In general, I think that if you are enforcing sections, then the field should be required, which is an easier way to fix this issue on a site.

Even with the new MR, this still needs:

* A test
* Some code cleanup

🇺🇸United States agentrickard Georgia (US)

There is a lot to digest here and it definitely means a 2.1.x branch.

IMO we should target this for Drupal 11.2 which is currently scheduled for June 2025

References related to changes in this PR:

* https://symfony.com/doc/current/service_container/autowiring.html
* https://phpstan.org/writing-php-code/phpdocs-basics#prefixed-tags
* https://php.watch/versions/8.1/readonly
* https://www.drupal.org/about/core/policies/core-release-cycles/schedule#...

🇺🇸United States agentrickard Georgia (US)

I don't think this is the right place to make this change, as it changes AccessControlHierarchyInterface (though that is not included in the patch).

If you have a one-site use-case, the recommended solution is to extend the Plugin with your own replacement and use that instead. See the Access Control section of the README.

This article (which includes some salty language) shows how to do that.

https://medium.com/@djphenaproxima/how-to-bend-drupal-8-plugins-to-your-...

🇺🇸United States agentrickard Georgia (US)

Converts the JS to vanilla JS and removes all dependencies.

To test on Tugboat

1. Configure an editorial scheme using the Workbench Access vocabulary
2. Go to the Users page for a section (e.g. "/admin/config/workflow/workbench_access/section/sections/workbench_access/users")
3. Click the "Switch between textarea/autocomplete" link under the autocomplete / text area
4. On click the visibility of the autocomplete / text area should toggle

🇺🇸United States agentrickard Georgia (US)

I am still not used to comments on MRs, so tickets need updating,.

🇺🇸United States agentrickard Georgia (US)

Tagging with a related issue which may be the solution.

🇺🇸United States agentrickard Georgia (US)

No reviews? Likely going to commit.

🇺🇸United States agentrickard Georgia (US)

I wonder if this is a better fit for Workbench Menu Access?

If we left it here, I would suggest that we also do that for Taxonomy overviews and make it part of the plugin API.

🇺🇸United States agentrickard Georgia (US)

Marking for review.

🇺🇸United States agentrickard Georgia (US)

Committed to 2.0.x

🇺🇸United States agentrickard Georgia (US)

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

🇺🇸United States agentrickard Georgia (US)

Yeah,. I thought we could set that to strict. I suppose we can only skip them.

🇺🇸United States agentrickard Georgia (US)

This appears to be a custom theme issue.

🇺🇸United States agentrickard Georgia (US)

Odd, I would have expected the MR to not have a green check in that case.

Anyway, fixed.

🇺🇸United States agentrickard Georgia (US)

MR is ready for review.

🇺🇸United States agentrickard Georgia (US)

This is working fine on a clean install, but when applied to an existing Drupal 11 site, the `Use header` option still appears.

Investigating.

🇺🇸United States agentrickard Georgia (US)

Patch no longer applies cleanly.

🇺🇸United States agentrickard Georgia (US)

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

🇺🇸United States agentrickard Georgia (US)

Here's a patch. I will fire up a MR in a minute.

However, it looks like this path only treats the symptom. The real issue is that in a preview context, some of the expected variables seem to be missing.

🇺🇸United States agentrickard Georgia (US)

Also seems to occur in 3.0.x

🇺🇸United States agentrickard Georgia (US)

agentrickard changed the visibility of the branch 3505640-create-drupal-11 to hidden.

🇺🇸United States agentrickard Georgia (US)

agentrickard changed the visibility of the branch 3505640-create-drupal-11 to hidden.

🇺🇸United States agentrickard Georgia (US)

agentrickard created an issue.

🇺🇸United States agentrickard Georgia (US)

Flagging, as I am interested in picking this back up, time permitting.

🇺🇸United States agentrickard Georgia (US)

I do not think that is valid. It is an abuse of the API. If anything, it should be caught upstream before hook_node_grants is called.

Also, the link to the code does not seem relevant to this issue. That code seems to have moved https://git.drupalcode.org/project/search_api/-/blob/8.x-1.x/src/Plugin/...

🇺🇸United States agentrickard Georgia (US)

Odd, the package redirecting to the new host -- https://packagist.org/packages/squizlabs/php_codesniffer

    "require": {
        "php": ">=7.2",
        "ext-mbstring": "*",
        "dealerdirect/phpcodesniffer-composer-installer": "^0.7.1 || ^1.0.0",
        "sirbrillig/phpcs-variable-analysis": "^2.11.7",
        "slevomat/coding-standard": "^8.11",
        "squizlabs/php_codesniffer": "^3.11.3",
        "symfony/yaml": ">=3.4.0"
    },

🇺🇸United States agentrickard Georgia (US)

agentrickard created an issue.

🇺🇸United States agentrickard Georgia (US)

@siramsay, I assume you are talking about this part "Create the directory core/themes/olivero/components/chip/"?

Why don't you go ahead and update that reference to assume an `example` theme installed in `themes/contrib/example`.

🇺🇸United States agentrickard Georgia (US)

I added an MR -- https://git.drupalcode.org/project/simple_sitemap/-/merge_requests/119 -- and updated tests.

Something is a bit off in the test. I can't seem to get the sitemap to re-render. See this part of the test code:

    // Test sitemap result -- Not working as expected.
    # $this->generator->rebuildQueue();
    # $this->generator->generate(QueueWorker::GENERATE_TYPE_BACKEND);
    # $this->drupalGet($this->defaultSitemapUrl);
    # $this->assertSession()->responseContains('node/' . $new_node1->id());
    # $this->assertSession()->responseNotContains('node/' . $new_node2->id());

This returns a 404 instead of the sitemap. I assume that I missed a step somewhere.

🇺🇸United States agentrickard Georgia (US)

Here's a version of the patch against 4.x. It does the following:

* Removes the form alter that disables per-entity overrides.
* Adds a method getAllowedEntities() to the EntityManager service
* Call that method from the EntityUrlGenerator in cases where the bundle is not auto-indexed.

Based on my reading of the code, access checking is handled by EntityUrlGeneratorBase, so that is omitted.

This likely needs a test, though is seems to work in my use-case.

If you would prefer an MR, I can make one.

🇺🇸United States agentrickard Georgia (US)

I can confirm the same problem on Platform.sh (Solr 9.6) using the provided configset from the module.

Putting the provided configset in .platform/configsets/solr9/ throws the error.

https://solr.apache.org/guide/solr/9_6/configuration-guide/config-sets.h... suggests that the files should be in a nested /conf directory inside the configset.

When we move the files to a nested /conf directory (.platform/configsets/solr9/conf), the initial error is replaced by "Your config-set contains manually added customizations. Be aware that these will be lost when the config-set needs to be regenerated."

🇺🇸United States agentrickard Georgia (US)

For the OP, I ran into this as well and could not get Brightcove's iFrame to work, so I ended up with this:

In mytheme.theme.

/**
 * Implements hook_preprocess_brightcove_player_responsive()
 *
 * We cannot render Brightcove inside CKEditor.
 */
function mytheme_theme_preprocess_brightcove_player_responsive(&$variables) {
  $variables['preview'] = FALSE;
  $route = \Drupal::routeMatch();
  if ($route instanceof RouteMatchInterface && $route->getRouteName() === 'embed.preview') {
    $variables['preview'] = TRUE;
  }
}

In mytheme/templates/brightcove/brightcove-player-responsive.html.twig


{% if preview %}
  {% set img_url = '/' ~ directory ~ '/imgs/video-thumbnail-.jpg' %}

  <div class="brightcove-player sizing-responsive">
    <div style="max-width: {{- max_width ~ units -}};">
      <img class="video__thumbnail" src="{{ img_url }}">
    </div>
  </div>
{% else %}
  <div class="brightcove-player sizing-responsive">
    <div style="max-width: {{- max_width ~ units -}};">
      <video-js class="vjs-fluid"
                controls=""
                data-account="{{- account -}}"
                data-embed="{{- embed -}}"
                data-player="{{- player -}}"
                data-usage="{{- data_usage -}}"
                data-{{- type -}}-id="{{- id -}}"
      ></video-js>
    </div>
    {% if is_playlist %}
      <ol class="vjs-playlist"></ol>
    {% endif %}
    <script src="//players.brightcove.net/{{- account -}}/{{- player -}}_{{- embed -}}/index.min.js"></script>
  </div>
{% endif %}

🇺🇸United States agentrickard Georgia (US)

agentrickard created an issue.

🇺🇸United States agentrickard Georgia (US)

This MR needs to pull in the changes from 📌 Fix failing test Fixed

🇺🇸United States agentrickard Georgia (US)

Latest patch is working nicely on Drupal 10.3.6.. Both import and export of a Layout Builder page work as expected.

🇺🇸United States agentrickard Georgia (US)

So you filed an MR that requires 10.3+ and didn't note that at all until after it was committed?

The current published release is now 1.6 and tests on GitLab CI with 10.3.

🇺🇸United States agentrickard Georgia (US)

I will also note that I reviewed the module and it has no API and no clear means for adding items.

I would wait until it is stable and those are provided, as they are in Toolbar now.

🇺🇸United States agentrickard Georgia (US)

I am going to remove the update hook because it has no added value to a site.

Any site that had an error due to basic_html would have already corrected it. New installs should be fine with the new config.

🇺🇸United States agentrickard Georgia (US)

Sadly, the update hook breaks any customizations and this needs to be revisited.

🇺🇸United States agentrickard Georgia (US)

8.x-1.x should be 10.3 compatible.

I am testing on Drupal 11 and the dev branch works fine.

I'll need a new issue if DEV isn't working on 10.3, with specific errors.

🇺🇸United States agentrickard Georgia (US)

Are you able to test the dev branch to ensure it will be stable?

🇺🇸United States agentrickard Georgia (US)
🇺🇸United States agentrickard Georgia (US)

This is being worked as part of 📌 Fix failing tests Active

🇺🇸United States agentrickard Georgia (US)

Still marked as experimental in 11.x

https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/navig...

I'd be happy to review an MR though.

🇺🇸United States agentrickard Georgia (US)

agentrickard created an issue.

🇺🇸United States agentrickard Georgia (US)

Fixed up tested and committed.

Nice work on the update hook.

🇺🇸United States agentrickard Georgia (US)

Update hooks go in module .install files.

🇺🇸United States agentrickard Georgia (US)

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

🇺🇸United States agentrickard Georgia (US)

Toolbar is a core feature.

🇺🇸United States agentrickard Georgia (US)

The open question is how this logo might fit with the other "Workbench" projects, which are listed here: https://www.drupal.org/project/project_browser/issues/3446128 📌 Logos for agentrickard Active

🇺🇸United States agentrickard Georgia (US)

This was an error when updating code style.

🇺🇸United States agentrickard Georgia (US)

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

🇺🇸United States agentrickard Georgia (US)

agentrickard created an issue.

Production build 0.71.5 2024