πŸ‡·πŸ‡΄Romania @andreic

Account created on 4 May 2006, about 18 years ago
#

Recent comments

πŸ‡·πŸ‡΄Romania andreic

As luck would have it, my VPS where the website is hosted got a system failure before that particular article was backed up.
Nevertheless, I recreated it and I updated the link above.

πŸ‡·πŸ‡΄Romania andreic

I found myself getting blocked by this error on a Drupal 9 to 10 upgrade and I was able to fix it. I documented the process here https://condurachi.ro/unknown-timediff-filter-twigexpressionparser-getfi....

πŸ‡·πŸ‡΄Romania andreic

Sorry, I'm not sure I understand.
All I did was add a checkbox in the View, which disables the double click action (screenshot attached).
That's what my client needed. Not sure what drop down box you mean.

πŸ‡·πŸ‡΄Romania andreic

Never mind. I was doing a ksort() on hook_form_alter() to find the form keys easier. Apparently that doesn't sit well with Form API.
All fixed.

πŸ‡·πŸ‡΄Romania andreic

I've set this to Critical because the module is unusable.

πŸ‡·πŸ‡΄Romania andreic

The patch in #55 fixed the same error I had on a 10.1.7 installation.

πŸ‡·πŸ‡΄Romania andreic

andreic β†’ made their first commit to this issue’s fork.

πŸ‡·πŸ‡΄Romania andreic

Works fine and fixes the error on Drupal 10.1.5.

πŸ‡·πŸ‡΄Romania andreic

I can confirm the patch works fine.

πŸ‡·πŸ‡΄Romania andreic

The existing code takes care of full URL already. Check out src/UnsplashFetcher.php line 54.

πŸ‡·πŸ‡΄Romania andreic

I've just tested with a clean install of 10.1.1 and when accessing admin/structure/taxonomy/manage/tags/overview it does not display the error.
It must be something with that particular client.

πŸ‡·πŸ‡΄Romania andreic

If I go to admin/structure/taxonomy/manage/tags/overview I also get it:

Deprecated function: str_starts_with(): Passing null to parameter #1 ($haystack) of type string is deprecated in Drupal\path_alias\PathProcessor\AliasPathProcessor->processOutbound() (line 54 of core/modules/path_alias/src/PathProcessor/AliasPathProcessor.php).

πŸ‡·πŸ‡΄Romania andreic

This is weird, I'm getting the error when accessing paths like taxonomy/term/327.
I've upgraded to 10.1.1 and now I get:

Deprecated function: str_starts_with(): Passing null to parameter #1 ($haystack) of type string is deprecated in Drupal\path_alias\PathProcessor\AliasPathProcessor->processOutbound() (line 54 of core/modules/path_alias/src/PathProcessor/AliasPathProcessor.php).

πŸ‡·πŸ‡΄Romania andreic

Patch file attached. Must get applied after https://www.drupal.org/project/media_entity_unsplash/issues/3373586 ✨ Allow adding by photo URL Needs work .

πŸ‡·πŸ‡΄Romania andreic

After applying the patch to the latest module version with composer.json, I fixed all the remaining errors when it gets applied and finally it's ready to use.

πŸ‡·πŸ‡΄Romania andreic

I've updated the patch to include only the State API changes.

πŸ‡·πŸ‡΄Romania andreic

It works, that fixed it!

πŸ‡·πŸ‡΄Romania andreic

1. Must be something I overlooked.

2. Well, first, the keys should not be in config/install/media.type.unsplash.yml file because that goes to Config. Then, some helper functions (e.g. unsplashGetKey and others) and then at the end, changes for the unsplash library because the library does not use access_key or secret_key, it uses applicationId, secret and utmSource as required keys when making call.

3. The links you gave offer the same solution: State API, which is what I'm doing, too.

πŸ‡·πŸ‡΄Romania andreic

Updated patch for all exceptions with appropriate message/hint.

πŸ‡·πŸ‡΄Romania andreic

Actually, you were perfectly right: "The client column in the db is NULL for all of them.".
I've encountered this myself and after a few hours of debugging, the problem was in the src/Normalizer/RefreshTokenEntityNormalizer.php file. It didn't include the client ID.
I've attached a patch.

πŸ‡·πŸ‡΄Romania andreic

#3 works for me. thanks.

Production build 0.69.0 2024