Account created on 4 February 2008, almost 17 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States nsciacca

In addition to the issues above we noticed the that global roles that were assigned to edit one content type could no longer edit nodes that were in groups. For example:

1. Global (Outsider) role of "Person Administrator" (PA) assigned to edit all nodes of content type of "Person"
2. After upgrade to 3.3 the PA could only edit Person nodes that were NOT in groups
3. Disabling the group_support_revisions module allowed them to edit all nodes of Person again (expected/previous behavior)

🇺🇸United States nsciacca

FYI - in case anyone lands here - I re-opened a new issue and submitted a MR here: https://www.drupal.org/project/quiz/issues/3485142 🐛 DivisionByZero error Active

🇺🇸United States nsciacca

nsciacca created an issue.

🇺🇸United States nsciacca

Created a MR for this. The score function within MatchingResponse needed a setEvaluated flag and there was a small change to the penalty choice logic since unchosen optino had a value of "" instead of "-1"

🇺🇸United States nsciacca

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

🇺🇸United States nsciacca

I moved nguyenphan's patch into a MR based on the latest branch. This fixes the issue for me and the answers are saved. I'm still seeing an issue with the total score not being calculated but I think there's a separate issue for that I will review.

🇺🇸United States nsciacca

Created a MR for the 4.x branch and removing patches from this issue.

🇺🇸United States nsciacca

I've been communicating with Pantheon's support staff and they've said:

Good day, for an update, Pantheon Platform Engineers already released an update to the Search API Pantheon module https://www.drupal.org/project/search_api_pantheon just today. The latest version 8.1.7 contains fixes for the Sticky Solr Schema.

I'll be installing this and monitoring on site asap.

🇺🇸United States nsciacca

Update: one of our sites randomly reverted back to 4.2.10 on 3/1 even though there was no deploy. This is a huge problem as it prevents new items from being indexed and there's no notice or warning other than our customers reporting missing content.

🇺🇸United States nsciacca

FYI - I did something similar in another ticket (3337669) for files that worked nicely using their pantheon_clear_edge_paths function and passing it the file url when the file was updated. In conjunction with the Media Entity File Replace module it seems to work well.

https://www.drupal.org/project/pantheon_advanced_page_cache/issues/3337669 🐛 Not showing newly updated/changed file on Pantheon - Cache issue Needs review

https://git.drupalcode.org/project/pantheon_advanced_page_cache/-/merge_...

🇺🇸United States nsciacca

I opened another MR for the 10.2.x branch as neither existing MR/patch was applying cleanly. Includes my modification with the extra selectors that were breaking Entity Browser.

🇺🇸United States nsciacca

@robbdavis - If you've previously patched core for the same issue I have noted above, try the patch from the MR over on https://www.drupal.org/project/drupal/issues/2858890#comment-15337125 🐛 Drupal.views.ajaxView is not initializing pagers in nested views Needs work

I ended up making some adjustments there and it helped.

https://git.drupalcode.org/project/drupal/-/merge_requests/4242/diffs.patch

🇺🇸United States nsciacca

@joseph.olstad Thanks for the reply but what's interesting is that I don't have Inline Form Errors enabled. There are no javascript errors thrown and nothing on the PHP side in the watchdog logs. Here's a video of me trying to select something:

https://www.dropbox.com/scl/fi/zr0lhshhkzl4cunjuf2lm/Entity-Browser-AJAX...

I guess there could be something else in our custom code that could be conflicting so I will try to spin up a vanilla Drupal with similar config to see if I can replicate it there.

🇺🇸United States nsciacca

Thanks for this! I added a couple other parameters for YouTube when it's used as background image to suppress the controls, set the loop and a playlist param so it continues to play.

🇺🇸United States nsciacca

Here's an updated patch for Drupal 10:

- Updated to simplify theme functions into .module
- Replaced core/jquery.once with core/once
- Replace file_munge_filename
- Replace usage of $_POST with request_stack service
- Drupal 10 code standards

Also there seems to be activity/work on the other issue here: Automated Drupal 10 compatibility fixes
- https://www.drupal.org/project/dropzonejs_webform/issues/3296994

🇺🇸United States nsciacca

The patch in #91 no longer applies to 3.x-dev and it's not an easy re-roll... the split of modes with Basic, Intermediate, and Advanced complicates this.

🇺🇸United States nsciacca

+1 for the patch. I didn't have any issue with the browser back as reported by arturs.v

🇺🇸United States nsciacca

Thanks @Wilfred Waltman - patch helped me when the View failed to display to logged in users.

🇺🇸United States nsciacca

New patch attached - the first one didn't account for multiple misspelled terms.

🇺🇸United States nsciacca

Thanks for the re-roll, working well for Drupal 9.5.10 + Facets 2.0.6 + Views Ajax History 1.7.0

🇺🇸United States nsciacca

I got a similar error with Views Bulk Operations:

Error: Call to undefined method Drupal\views_bulk_operations\Form\ConfigureAction::getEntity() in Drupal\rabbit_hole\Plugin\Field\FieldWidget\RabbitHoleDefaultWidget->formElement() (line 168 of modules/contrib/rabbit_hole/src/Plugin/Field/FieldWidget/RabbitHoleDefaultWidget.php).

Adding the below check to the top of the formElement function works for my instance.

if (!method_exists($form_state->getFormObject(), 'getEntity')) {
   return;
}
🇺🇸United States nsciacca

I can confirm this bug - I was just about to log it as well. It happens in both D9 & D10 with CKEditor5. I'm not sure where the fix needs to be... but it's the source that doesn't get updated. If you update the image and switch to view the source you'll see the old image url in the "src" attribute, so when you switch back it shows the original image.

🇺🇸United States nsciacca

+1 this patch worked for me

🇺🇸United States nsciacca

Updated patch for D10 as #66 had a duped line.

🇺🇸United States nsciacca

@GuStAvE33 Here's a patch for D10 using core/once instead of jquery once which is likely what was failing.

I was able to get this to work as desired with the patch but I had to enable the facets_summary block, which I wasn't using previously. The JS library was added to that submodule, but I don't think it needs to be there in the long run. I was able to enable it and just put a button with class "facets-apply-button" on the page with my existing checkbox facets that were updated to not auto-submit and it worked.

🇺🇸United States nsciacca

+1 for this patch. Just ran across this in the newer version of Fullcalendar View when upgrading to Drupal 10 and this saved the day. Thanks!

🇺🇸United States nsciacca

Patch in #32 worked for me. I had Layout Builder blocks referencing paragraphs and found that cloned nodes that were unpublished were setting the source node's paragraphs to be unpublished - so anonymous visitors would see a blank page visiting the source node. This is working, thank you.

🇺🇸United States nsciacca

Is it possible to get these changes into a dev branch? Currently you cannot install 2.0.0 and then patch it as composer will throw requirement conflicts.

🇺🇸United States nsciacca

@mherchel - I just applied this to my 10.1 and other than a warning "] rmdir(/files/css): Directory not empty FileSystem.php:267" it did appear to fix the aggregate CSS/JS on my Pantheon deploy.

🇺🇸United States nsciacca

I think this should be re-opened, I tried with the latest version and found the following issues:

1. The actions don't exist for the Bulk Form for media. I manually copied the "system.action.publish_latest" and "system.action.unpublish_latest" to media specific ones to get this to work from within my config sync. However, there should be examples in the module's config/optional folder.

2. Both action plugins for "PublishLatestRevisionAction" and "UnpublishCurrentRevisionAction" had checks in the access function for whether the entity was a node, therefore it would return an Access Denied when applying the action. I updated this to check for either nodes or media.

3. There should be similar settings for the module for "Enable dialog on media edit form" and "Enable dialog on admin media listing". This affects the dialog pop ups and the code is all hard coded to look for node forms.

I have attached a patch for 1 & 2 which at least makes the module functional for media entities. #3 is a larger lift I don't have time for at the moment.

🇺🇸United States nsciacca

The patch fixes the error from breaking the page, however the functionality is broken. Not sure if they're related but this issue describes the context that's not working: https://www.drupal.org/project/term_condition/issues/3363916 🐛 2.0.2 breaks backward compatibility Fixed

🇺🇸United States nsciacca

This didn't work for me, the tokens can't be enclosed in quotes.. not sure if it's a newer database incompatibility. Anyhow, this is a super old issue but in case anyone else runs across it, here's an updated patch

🇺🇸United States nsciacca

I couldn't apply the patch to 2.1 when using composer to manage patches, so I re-created it. Attaching in case anyone else needs.

🇺🇸United States nsciacca

I can no longer reproduce the error. I updated back to 2.1.1 and it's connecting fine, even without the patch added. Strange, but I think this is good to go.

🇺🇸United States nsciacca

@smustgrave I think my issue is actually caused by the change in https://www.drupal.org/project/drupal/issues/3294720 🐛 The attachBehaviors() for document is only called after Big Pipe chunks are processed Fixed . As that change was the one that introduced another attachBehaviors call whereas this patch just modified an existing call. Okay to move forward, and thanks for checking in.

🇺🇸United States nsciacca

Retracting my comment above. Using version 3.1.3 and this no longer is an issue.

Production build 0.71.5 2024