🇺🇸United States @pmagunia

I was getting a JavaScript SHS App model error on Drupal 11 with 2.0.0.

When I reverted to 2.0.0-rc12 and cleared the cache it fixed the issue.

The last patch couldn't be applied to Drupal 10.3.9.

Here is a reroll.

Why would you want to feed tokens into R?

Is there a use case for it?

Please feel free to re-open if necessary.

New versions of the module have been created that support Drupal 10 and 11.

pmagunia → created an issue.

Sorry it has taken me awhile to get back to you.

With the newer versions you can use Drush to download the library automatically to the correct location.

Please feel free to re-open if necessary.

Going to close this issue since it's been awhile since it has been created and is most likely an issue with the upstream jsDraw2d library.

Please feel free to re-open this issue if you are still having issue.

Closing it for now.

If you have access to Drush you can do drush whiteboard:download. It's listed in the README.

Otherwise the structure should look like the image.

Or /libraries/jsdraw2d/jsdraw2d.js

If you're still having issues, please let me know.

@metasim I don't think you need the patch you supplied.

Just go to the Reroute Email settings and click save.

That will update the config.

Added a sentence about using the ddev exec command instead of DDEV SSH.

Sorry for my absence. Thanks for the comments and work so far.

I made a commit that includes the work in this thread and gave everybody issue credit.

It does need testing. Please use the 2.0.x-dev branch for now.

Drupal 9 is no longer supported so marking this as Won't Fix.

We may want to consider switching from MarkItUp to CKEditor5.

pmagunia → created an issue.

Fixed with:

https://www.drupal.org/project/r/issues/3486505 →

pmagunia → created an issue.

This original thread was meant for other maintainer who took over the project.

It looks like has not maintained the project.

Taking over as maintainer.

You can follow this thread to know when a Drupal 10 version will be released.

https://www.drupal.org/project/r/issues/2690871 →

It looks like the other maintainer has not maintained the project.

I am going to restart maintaining it.

Tokens are not currently supported but patches are welcome.

It looks like that library is no longer maintained.

I'm not sure this is an issue anymore.

Please feel free to open another thread if it is.

Here is the Drupal 10 version issue page. You can follow the issue to be notified when it has been updated.

https://www.drupal.org/project/r/issues/3485229 →

Sorry it has taken me awhile to get back to you.

Drupal 9 is no longer supported but I've been working on a Drupal 10/11 version.

I will try to release that in the next couple of weeks. I've got the code ported I'm just trying to create the tests for the module.

Sorry it has taken me awhile to get back to you.

This might seem like a silly question but do you have R installed on your webserver?

pmagunia → created an issue.

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

Thanks @mstrelan.

I just corrected my previous commit.

pmagunia → created an issue.

It looks like an Entity Query is being made without an AccessCheck:

https://git.drupalcode.org/project/openid_connect/-/blob/3.x/src/Form/OpenIDConnectClientFormBase.php?ref_type=heads#L133

Created branch 3431913-automated-drupal-11 with added newline and Drupal 11 Compatibility.

Upgrade Status module didn't report any other issues.

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

pmagunia → created an issue.

Thank you for reviewing

I don't think there is an upgrade path from v3 according to the module's homepage.

There are some submodules in this module that need core version updated.

pmagunia → created an issue.

The patch in Comment #4 no longer applies to 10.3.0.

Trying a reroll. Not sure it works

This may be the module you are looking for if you have a premium subscription to Twitter:

https://www.drupal.org/project/twitter_api_block →

I guess it may be worth adding a message on this module's front-page that Twitter requires a premium paid subscription so users are aware for transparency.

I don't plan to get a premium subscription either.

pmagunia → created an issue.

This is still an issue that is holding us back.

I think they made the Twitter API a premium feature that requires a credit card.

Even after the patch in #41 Commerce Recurring fails upon uninstall for me:

commerce_recurring: The <em class="placeholder">Billing period</em> field type is used in the following fields: commerce_order.billing_period, commerce_order_item.billing_period

I haven't had much luck uninstalling Commerce Core either.

For Drupal 10 and Commerce v2.39, I had to do something slightly different:

/**
 * Implements hook_commerce_order_update().
 */
function MYMODULE_commerce_order_update($order) {
  if ($order->getState()->getId() == 'completed') {
    // Business logic like adding a role or dpm().
  }
}

I know this post is for D7 but thought I'd add the D10 version since its a top hit.

<?php

/**
 * @file
 * Contains MYMODULE.module.
 */

/** 
 * Implements hook_form_alter().
 */
function MYMODULE_form_alter(&$form, &$form_state, $form_id) {
  // Switch statement may be better if you have a lot of options.
  if ($form_id == 'MYFORM') {
    // Use dpm($form_id) if you are unsure of the form's ID.
    // The following statement may need to be adjust depending on your field type.
    unset($form['field_MYFIELD']['widget']['#options']['_none']);
  }
}

We're using the Tabs field group and have "Mark group as required if it contains required fields" checked.

On the edit page there are now 2 red asterisks in the tab that has a required field. Not sure if that is expected behavior.

The patch from #52 applied cleanly to Drupal 10.2.5.

Thanks for getting back so quickly.

A client on a Windows machine was having difficulty opening the file.

pmagunia → created an issue.

👍

The placeholders change made by jedihe caused SQLSTATE[42S22]: Column not found: 1054 Unknown column

I went back to the old code and it worked.

I was having the same issue. The solution in #8 fixed it for me. Thank you Fernly.

I think style attributes are not allowed in "Limit HTML" filter.

It also seems Word comments are not working elegantly. I tried to upload the docx sample file I used but the file extension is blacklisted.

I was using Safari 17.1.2

You're welcome Wim. I was using Safari 17.1.2. I did mention that in the Github thread you provided in comment #26

I copied the sample Google Doc into the nightly CKE5 textarea posted by Reinmar and still saw the issue:

Adding a screen recording of the issue

pmagunia → created an issue.

To minimize permission errors, I chose to set accessCheck to FALSE.

Providing a patch for that

The MR below comment #25 fixed the issue for our use case.

This issue is reproducible in Drupal 10.1 also.

If someone wanted to use the code in the patch:

function MYMODULE_update_10000(&$sandbox) {
  \Drupal::service('entity_field.manager')->rebuildBundleFieldMap();
}

Adding a file patch in case anyone want to link. Identical to #38

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

Thanks for the suggestion @occupant.

I haven't really had time to investigate thoroughly but it could be.

I will keep that in mind.

I just tried it on simplytestme and it seems to be working OK.

I guess its an issue with our system.

pmagunia → created an issue.

I couldn't upgrade to admin_toolbar to the latest version due to some dependencies.

I thought it would be easier to patch here.

Not sure its acceptable.

Thanks for the quick reply.

We are using admin_toolbar 3.3.2 and Drupal 9.5.10.

I see this line in the older version of the admin_toolbar we are using:

#toolbar-bar {
  z-index: 100000;
}

https://git.drupalcode.org/project/admin_toolbar/-/blob/3.3.2/css/admin....

Our Drupal report is not explicitly saying its no longer supported, but there is a newer version on the Drupal.org admin_toolbar page.

pmagunia → created an issue.