Account created on 12 May 2016, over 8 years ago
  • Drupal Developer at VDABΒ 
  • Technical Designer at CegekaΒ 
#

Merge Requests

Recent comments

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

@agoradesign Thx! I fixed this and also added an accessCheck I missed on the entityQuery in the WebPushNotificationController in commit 112133ad

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

I created a D10 compatible branch that we are currently using on our project in order to upgrade to D10.

Current changes on that branch can be reviewed already.

In the meantime people that are using this module can use the 3393651-d10-compatibility branch to upgrade their site to D10.

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

We required this feature when upgrading to Drupal 10.1.2.

I rerolled the patch for use in drupal 10.1.x

Patch might not apply for 10.1.2 specifically. In that case, checkout the 10.1.2 core tag, apply the patch and if the hunk on
drupalMedia.js fails to apply, you can skip that and remake the patch after running yarn in the core folder:

cd core
yarn install
yarn build:ckeditor5

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

@octaviosch Like @Nathaniel mentioned there is still a deprecated hook used (image_field_caption_field_widget_form_alter), I changed
this to image_field_caption_field_widget_single_element_form_alter and commited this on the 3355337-support-drupal-10 branch.
I'll also attach a remake of the existing patch with this change included.

The caption field should be usable now.

I'll mark this as needs review.

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

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

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

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

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

We had an issue with the latest patch,

In our case, the strings from RfcLogLevel::getLevels() were translated, which resulted in a Call to undefined method Drupal\Core\Logger\LoggerChannel::***()

I fixed this in the latest patch and also added the comment suggestion from #10

      // A severity of -1 is not a valid RfcLogLevel but is the default value. With -1,
      // Ultimate Cron logs a message saying that cron has been executed, which is
      // unnecessary if you're running cron every minute as recommended.
      if ($log_entry->severity > -1) {
        $severityLevels = RfcLogLevel::getLevels();
        $severity = strtolower($severityLevels[$log_entry->severity]->getUntranslatedString());
        $this->loggerFactory->get('ultimate_cron_' . $log_entry->name)->{$severity}($log_entry->message ?: $log_entry->formatInitMessage());
      }
πŸ‡§πŸ‡ͺBelgium jOpdebeeck

Remade the patch from #48

Re-rolled #40 for 9.4.12

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

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

πŸ‡§πŸ‡ͺBelgium jOpdebeeck

The block ui library repository needs to be added to your composer.json file:

Installation:
Add both this module and the block ui library to your project's composer.json, which is a dependency of this module:

```
"repositories": [
{
"type": "package",
"package": {
"name": "library-blockui/blockui",
"version": "v2.70",
"type": "drupal-library",
"dist": {
"url": "https://github.com/malsup/blockui/archive/2.70.zip",
"type": "zip"
}
}
}
]
```
Install it, as usual, with: `composer require drupal/elasticsearch_search_api`

Production build 0.71.5 2024