🇭🇺Hungary @kecsot

Hungary
Account created on 26 October 2013, about 11 years ago
#

Recent comments

🇭🇺Hungary kecsot Hungary

ParagraphAccessControlHandler::checkAccess

    if ($paragraph->getParentEntity() != NULL) {
      // Delete permission on the paragraph, should just depend on 'update'
      // access permissions on the parent.
      $operation = ($operation == 'delete') ? 'update' : $operation;
      // Library items have no support for parent entity access checking.
      if ($paragraph->getParentEntity()->getEntityTypeId() != 'paragraphs_library_item') {
        $parent_access = $paragraph->getParentEntity()->access($operation, $account, TRUE);
        $access_result = $access_result->andIf($parent_access);
      }
    }

The parentEntity->access was only "neutral". The anonymous user not had access to rendered entity.

Works as designed. Sorry :)

🇭🇺Hungary kecsot Hungary

looks like permission issue. it works with administrator role but not with anonymous

🇭🇺Hungary kecsot Hungary

Great idea.

I've tested and works well with 10.2.7 and 10.3

Thank you!

🇭🇺Hungary kecsot Hungary

I see it's fixed in v2.0.0.
I've updated.

The problem is v1.1.6 was compatible with 10.3 drupal.
Than my patch is not a solution.
We need to deny d10.3.0 from v1.x with composer.

🇭🇺Hungary kecsot Hungary

Are you sure?

* @phpstan-type AndroidConfigShape array{
...
*      ttl?: positive-int|non-empty-string|null,
 *     notification?: AndroidNotificationShape,
...
 * }
* @phpstan-type AndroidNotificationShape array{
...
 *     notification_count?: positive-int,
...
 * }
    /**
     * @param AndroidConfigShape $config
     *
     * @throws InvalidArgument
     */
    public static function fromArray(array $config): self
    {
        if (array_key_exists('ttl', $config) && $config['ttl'] !== null) {
            $config['ttl'] = self::ensureValidTtl($config['ttl']);
        }

        return new self($config);
    }

It looks like it should be wrapped.

Just like:

    public function withNotificationPriority(string $notificationPriority): self
    {
        $config = clone $this;

        $config->config['notification'] ??= [];
        $config->config['notification']['notification_priority'] = $notificationPriority;

        return $config;
    }
🇭🇺Hungary kecsot Hungary

You are looking for:

$config = AndroidConfig::fromArray([
    'notification' => [
        'notification_count' => $badge_count
    ]
]);
🇭🇺Hungary kecsot Hungary

Looks like an update hook missing if you update from D9 to D10.

🇭🇺Hungary kecsot Hungary

Thank you. I've created tests.

Do not revoke permission if another module grants for any reason.
I think PFDP should just grant more permission, but not remove any 3rd party permission.

🇭🇺Hungary kecsot Hungary

Code in MR3!

I'm open to change the view of shy char in ckeditor. I uploaded a screenshot from my version.

Production build 0.71.5 2024