Brooklyn, NY
Account created on 25 September 2008, about 17 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States jrockowitz Brooklyn, NY

I am getting beta4, and I am on Drupal 10.4.

What version of Drupal are you on, because the Drupal dependency for the CER module change between beta3 (^9.5 || ^10) to beta4 (^10.3 || ^11)

https://www.drupal.org/project/cer/releases/5.0.0-beta3
https://www.drupal.org/project/cer/releases/5.0.0-beta4

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

I thought "drupal/cer": "^5.0", allowed beta releases to be installed.

🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY

This module is still in alpha, and it is possible/likely that some. config did not get updated or exported as expected.

One workaround is to back up and restore the schemadotorg.mapping.*.yml files, with a reinstall.

🇺🇸United States jrockowitz Brooklyn, NY

I'm sure it is one of the many, many schema modules.

At the same time, you can start disabling modules and see if it cause the issue.

With no error message or log, I won't be able to debug this issue.

🇺🇸United States jrockowitz Brooklyn, NY

Amongst other things, it re-thinks some Drupal UIs, specifically form actions, that are moved to the top right corner.

I am seeing this impact other contrib modules including custom_field.module, but we should look into if there is way to fix the problem via Gin or a contrib modules

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

I am very confused by your example, you are nesting a flexbox inside a container, creating an unexpected layout. If you remove the container, the flexbox layout starts being more predictable. Also, uou are also mixing inline labels with the default vertical label.

I think you either have to write a little CSS to get the layout your desire or simplify your layout.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

Please provide an example webform which can be used to replicate this issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY

The MR looks good. I appreciate the specificity of the fix. Someone needs to do some manual review.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

I don't want to implement this feature because it would require too much work to support it.

For example, the webform views module would most likely need to be updated and tokens will probably need to fixed.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

Please reopen this ticket or create a new one if you can provide steps to replicate the issue.

🇺🇸United States jrockowitz Brooklyn, NY

Please reopen this ticket or create a new one if you can provide steps to replicate the issue.

🇺🇸United States jrockowitz Brooklyn, NY

Please reopen this ticket or create a new one if you can provide steps to replicate the issue.

🇺🇸United States jrockowitz Brooklyn, NY

Please reopen this ticket or create a new one if you can provide steps to replicate the issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

Please reopen this ticket or create a new one if you can provide steps to replicate the issue.

🇺🇸United States jrockowitz Brooklyn, NY

I am closing this issue but I did see something similar with. Custom code that added a callback using $this inside a service or OOP hook.

The below code will cause the $this (i.e, a service) to serialized.

$form['actions']['save']['#submit'][] = [$this, 'formSubmit'];

The code had to be changed to

$form['actions']['save']['#submit'][] = [static::class, 'formSubmit'];

🇺🇸United States jrockowitz Brooklyn, NY

We need to be very careful with our solution because the fieldgroup css is most likely going to be removed.
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/lib/Drupal/Co...

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

Tests are passing, and I'm confident that element access controls have sufficient test coverage.

🇺🇸United States jrockowitz Brooklyn, NY

Hmm I am almost would rather add the missing CSS to something like webform.form.css.

🇺🇸United States jrockowitz Brooklyn, NY

The webform add-ons service was removed. I think running update.php will resolve this issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

The "Schema.org Blueprints JSON-LD Preview" provides the tab

🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY

I think this feature can be part of the schemadotorg_jsonld_preview.module.

We should add the 'Schema.org' tab to all supported content entity types.

  • media
  • taxonomy
  • user
  • block
  • etc...

The routes and tasks would have become dynamically generated.
We do something similar to the devel.module.

  • \Drupal\devel\Routing\RouteSubscriber
  • \Drupal\devel\Plugin\Derivative\DevelLocalTask
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY

You can modify or create your own CSV file and define it via Schema.org data file/URL. (/admin/config/schemadotorg/settings/general)

@see Bioschemas or other schemas support Needs review

🇺🇸United States jrockowitz Brooklyn, NY

It make sense to make sure the $element['#webform'] is not empty via

I updated the MR to use the below code.

    // Make sure that the '#webform' property is defined before proceeding.
    if (empty($element['#webform'])) {
      return $element;
    }

If all tests pass, this should be RTBC.

🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz made their first commit to this issue’s fork.

🇺🇸United States jrockowitz Brooklyn, NY

Wow, forgot about this ticket and the patch, and just ran into the same issue via 🐛 PHP 8.1 TypeError: Drupal\webform\WebformRequest::getUrl() Closed: cannot reproduce and my conclusion is the the main entity (node) should always be stored as the source entity. Yet the problem is that tokens are current accessing the paragraph and node.

🇺🇸United States jrockowitz Brooklyn, NY

Yes, this is a challenging issue to resolve because when a webform is placed within a paragraph, the webform submission's source entity is the paragraph, but the webform node's source entity is the node.

The current MR might be best compromise

🇺🇸United States jrockowitz Brooklyn, NY

I was able to determine that source entity for the webform and submission are out-of-sync. I can't find the root cause.

The current MR is might resolve the issue but the root cause needs to be tracked down.

🇺🇸United States jrockowitz Brooklyn, NY

Here are the steps to reproduce this issues.

  • Create a node with paragraph containing a webform
  • Submit the webform
  • Edit the node and delete the paragraph, click Save.
  • Edit the node and re-add the same paragraph pointing to the same webform click save.
  • Confirm you get the TypeError: Drupal\webform\WebformRequest::getUrl(): Argument #1 ($webform_entity) error
🇺🇸United States jrockowitz Brooklyn, NY

Yeh, I was also looking into this and couldn't reproduce the bug.

🇺🇸United States jrockowitz Brooklyn, NY

Yep, the update hook is missing. I commited the fix the 1.0.x-dev.

🇺🇸United States jrockowitz Brooklyn, NY

Did you run `drush updb`?

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

RTBC if tests pass

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY

I got caught in my own update process. The key is to run the database updates via drush updb, which clears the service container, allowing hooks to call services.

🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

jrockowitz created an issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

The email confirm element's title is displayed via the email input and not the wrapper.

@see https://git.drupalcode.org/project/webform/-/blob/6.3.x/src/Element/Webf...

I should note, this works the opposite of how the "Address" component works. For the Address component,

This is because the address element's title is not used in the sub-elements.

The workaround if you need a fieldset visible around the email confirm element is to place the element inside a fieldset and use a simple container.

email_container:
  '#type': fieldset
  '#title': 'Enter your email'
  email:
    '#type': webform_email_confirm
    '#title': Email
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

Please reopen if you can provide an example webform similar to #11 that can reproduce this issue.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

Please reopen when you can provide the setup and steps required to reproduce this issue.

🇺🇸United States jrockowitz Brooklyn, NY

Please reopen when you can provide the setup and steps required to reproduce this issue.

🇺🇸United States jrockowitz Brooklyn, NY

Please reopen when you can provide the setup and steps required to reproduce this issue.

🇺🇸United States jrockowitz Brooklyn, NY

This needs an MR and should target 6.3.x

🇺🇸United States jrockowitz Brooklyn, NY

RTBC if tests are passing

🇺🇸United States jrockowitz Brooklyn, NY

I have merged the simple solution. Please reopen this ticket if there are any other problems.

🇺🇸United States jrockowitz Brooklyn, NY
🇺🇸United States jrockowitz Brooklyn, NY

RTBC assuming tests pass

🇺🇸United States jrockowitz Brooklyn, NY
Production build 0.71.5 2024