Account created on 8 January 2019, almost 6 years ago
  • Software Engineer at FloatΒ 
#

Merge Requests

Recent comments

There is also a reference to stable within the info.yml file. Here is a patch to map core's system theme to the (now contrib) seven theme. This should be applied alongside #12.

Hi jonathan1055,

The initial reasoning was to use a "safe" way to generate a random ID versus a potentially "unsafe" way. I did not want to change the functionality of what you originally had and I wanted to rely on Core as they have "best practices" for Drupal.

I dug a little more into Drupal's cron key generation in core/modules/system/system.install. They are using $cron_key = Crypt::randomBytesBase64(55);. Here is the original issue ( https://www.drupal.org/project/drupal/issues/2140433 β†’ ); rand is susceptible as the same "predictability" (https://www.php.net/manual/en/function.rand.php).

I've re-rolled the patch to follow Core.

Here is a patch for the current release.

@gisle We can improve the documentation to highlight the other modules. At a glance, here are the differences:

  1. Kaltura - This only supports D7
  2. Kaltura Media - This adds a [media] field specifically for Kaltura media
  3. Kaltura Video - This gives an interface to manage Kaltura media via Drupal using the Kaltura PHP SDK. It also provides a new entity type for videos
  4. Kaltura Video Embed Field - This is a [source] plugin for Video Embed, which allows users to embed videos to Drupal content (via a field)

@Michelle

Thank you for the update. I opted to exclude D8 since there was a drupal-check warning about return types.
I've pushed a new version (2.0.0).

Unfortunately, we have built our sites around switches and the other two modules would require additional work (versus just updating to D10).

@gisle Maybe with our help, we can grow the module. I'm a bit surprised that there are no installs (we are using the actively module on many sites). We could break this off to our own custom module, but would rather keep it open for the community to use.

I want to help maintain this module and resolve any issues in the queue, particularly by making this D10 compatible.

Thanks,
Brian

I updated the title.

I have not received any response from the maintainer.

I updated the title.

I have not received any response from the maintainer.

Here is another patch that adds a few return types as well as prefers dependency injection versus using `Drupal::service`

Thank you for this info. I will examine the open simple_oauth issue and see if our module will fit to cover this.

Production build 0.71.5 2024