🇮🇳India @ushma

By mistake uploaded wrong patch above. Please find below the correct patch for the issue.

Not able to reproduce..

I am using drupal10.3.6 and installed module version 3.0.1.
On visiting the text format pages, I dont see any warnings or errors before or after enabling the style filter.

I have created MR to resolve the warning by checking the array count - https://git.drupalcode.org/project/shield/-/merge_requests/24

Attached the related patch too. Please verify.

ushma → made their first commit to this issue’s fork.

I have created an MR to replace the deprecated method with removeDecoration(). Attached the patch for the same. Please review.

I have created MR to resolve this warning in case redirect_url remains undefined. - https://git.drupalcode.org/project/views_data_export/-/merge_requests/54

Attached related patch too. Please review.

I have created an MR to resolve this issue. Also attached related patch. Please review.

@hctom Added logic to handle empty response case. Also attached a patch for the same. Please review.

ushma → changed the visibility of the branch 10.x to active.

ushma → changed the visibility of the branch 10.x to hidden.

@mattbloomfield your welcome. It would be great if you provide credit in case the contribution seems useful to you.

The reason is- the current composer file of this module is restricting to fetch older version of Inline entity form module(which is a dependency here) and your project must be having the latest version..creating a conflict.
See this - https://www.drupal.org/project/brightcove/issues/3412590#comment-15408655 ✨ Allow the latest version of Inline Entity Form Needs review

Above patch resolves the issue for me.

Have you tried uninstalling the devel module and then running the composer update commad?

Please check if you are using the latest d10 compatible version devel module. The use of Twig_Extension is now removed in new symfony.

I have created a patch using the attched text file and also fixed the url issue. It will now show node alias instead of drupal internal path. Please verify this.

ushma → changed the visibility of the branch 3418597-window.onload-not-firing to hidden.

I "think" the issue is related to the following change - https://github.com/jquery/jquery/issues/3194 in jQuery 3.x where onload is not guaranteed to fire inside a $(Document).ready state any more - which is what Drupal.behaviours is giving you.

I tested with drupal 8.9, php 7.4.

I've made a patch and it works for me.

ushma → created an issue.

same...the fix #2 works on my D10.1.6

I am facing the same issue for drupal 10.1. Were you able to resolve it ?

Hi @eojthebrave
I am in procces to upgrade drupal 9.5 to drupal 10 and currently using recurly 4.0.0-alpha1
How can I make this module compatible right now since there's still no stable release of D10 compatible version. Please suggest.

Hi
I am able to save h5p content using above method but its not showing up properly in front end. I found out that there is no entry in h5p_content_libraries table for the new h5p content id created using H5PContent::create(). What am I missing here ?

Facing the same issue. Were you able to find out the actual reason for it ?

Hi KlementDEV
were you able to resolve the issue ?

If node access is forbidden then instead of hook_node_view_alter, you can use hook_preprocess_node to disable cache for for your specific node type

for example - 

function mymodule_preprocess_node(&$variables) {
  if ($variables['node']->getType() == 'your_content_type_machine_name') {
     $variables['#cache']['max-age'] = 0;
     \Drupal::service('page_cache_kill_switch')->trigger();
  }
}

fixed the error using elvis operator.We can avoid ternary operator here. and coalesce wont work in case $config is empty or 0.
Please review.

Hi webservant316
I am getting similar errors with drupal 9.4.5 and php 8.1. Were you able to fix them ?

Hi..is this MR working ?

I have fixed the issue. Please validate