Austin, TX, USA
Account created on 18 September 2006, about 18 years ago
#

Recent comments

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

@w01f Fair enough. I was riffing off of "Require registration to launch sites" in the issue summary.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

Not sure whether the capability exists yet, but I think it will be possible to integrate with drupal.org SSO.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

Copying the files changed by https://git.drupalcode.org/project/civictheme/-/merge_requests/7/diffs?c... into my sub-theme resolved the error for me.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

I ran into this as well while simply viewing a page node. It might have been using a sub-theme, but I hadn't made any changes to the sub-theme.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

Patch in #16 worked around the issue I was encountering: inability to create a user (without SAML) in a non-production environment that does not have the keys required by SAML.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

There's a patch that works around item 2 in #3 at https://www.drupal.org/project/samlauth/issues/2925171#comment-14904456 πŸ“Œ Use externalauth::register event vs hook_user_presave Needs work

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

Noting that the patch in #4 does not apply to 1.4.0

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

This was undone by πŸ› Drag to re-order doesn't work without autocomplete Fixed .

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

This added back the "Drag to re-order {entity_type}" help text that was removed in πŸ“Œ remove "Drag to re-order {entity_type}" Fixed . Was this intentional or unintentional?

Also related: ✨ Make it possible to disable reordering (drag & drop) Needs review

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

The patches are not yet relevant to Group 3.x.

I found that I had to do the following to programmatically add a group role for a user with an existing membership:

      // Get the user's membership in the group.
      $membership = $group->getMember($account);

      // If the user is already a member of the group.
      if ($membership) {
        /** @var \Drupal\group\GroupMembership $membership */

        $group_roles = $membership->getRoles(FALSE);

        $has_member_role = FALSE;

        foreach ($group_roles as $group_role) {
          if ($group_role->id() == $member_role) {
            $has_member_role = TRUE;
            break;
          }
        }

        // If the user doesn't already have the member role in this group.
        if (!$has_member_role) {
          // Add the member role for this user.
          $group_relationship = $membership->getGroupRelationship();
          $group_relationship->group_roles[] = $member_role;
          $group_relationship->save();
        }
      }
πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

This patch did not address the issue for us.

$now = REQUEST_TIME;

...

if (($now + self::REQUEST_TIMEOUT) < REQUEST_TIME) 

is equivalent to:
if (REQUEST_TIME + self::REQUEST_TIMEOUT < REQUEST_TIME)
which will always be false.

Perhaps also part of the problem is the use of REQUEST_TIME instead of time(). In the event that this code is reached from a long-running process (i.e. where the request started more than 60 seconds ago), the problem will be compounded.

Our use case involves operations performed by long-running Drush command executions though I don't believe that our niche use case is the only one that would be affected.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

#17 applied to 10.1.3 and allowed my view with aggregation to work with a Moderation State field.

Setting to Needs Review as there are no steps to reproduce the error described in #18 and this is working for me.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

Bug still exists in Group 3.2.0. #27 and #19 apply cleanly, but do not resolve the problem. Removing the group permission from the view's access control setting does, but of course that access restriction is there for a reason!

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

jdleonard β†’ created an issue.

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

Patch appears to work for me on Drupal 10.1.1. Thanks!

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

Cross-referencing Acquia support case #00975363

πŸ‡ΊπŸ‡ΈUnited States jdleonard Austin, TX, USA

#2 worked for me. I reported this at #3120168-5: No available releases found β†’ and was directed here. Thank you!

Production build 0.71.5 2024