bbeversdorf β created an issue.
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.
bbeversdorf β created an issue.
mediabounds β credited bbeversdorf β .
@gisle We can improve the documentation to highlight the other modules. At a glance, here are the differences:
- Kaltura - This only supports D7
- Kaltura Media - This adds a [media] field specifically for Kaltura media
- 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
- 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.
There has been no response in 14 days.
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.
bbeversdorf β created an issue.
Here is another patch that adds a few return types as well as prefers dependency injection versus using `Drupal::service`
bbeversdorf β created an issue.
bbeversdorf β created an issue.
The module has been updated.
bbeversdorf β made their first commit to this issueβs fork.
Thank you for this info. I will examine the open simple_oauth issue and see if our module will fit to cover this.
bbeversdorf β created an issue.