Asheville, NC
Account created on 12 October 2010, over 14 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States weekbeforenext Asheville, NC

Updated Asheville information.

🇺🇸United States weekbeforenext Asheville, NC

I resolved merge conflicts with the MR.

🇺🇸United States weekbeforenext Asheville, NC

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

🇺🇸United States weekbeforenext Asheville, NC

Invoice #1043 sent.
Payment transferred, Transaction ID: 23597630033
The invoice has been paid.

🇺🇸United States weekbeforenext Asheville, NC

Throwing some breakpoints in the area of conditional_fields.js under the comment "// Override state change handlers for dependents with special effects." and comparing between 4.0.0-alpha5 on Drupal core 10.2.11 and 4.0.0-alpha6 on Drupal core 10.3.10 I see a difference in the handler arguments.

If I put a breakpoint on `originalHander(e);` this is the comparison:

4.0.0-alpha5 on Drupal core 10.2.11 (working):

4.0.0-alpha6 on Drupal core 10.3.10 (not working):

The Type Error from the screenshot looks something like this:

TypeError: 'caller', 'callee', and 'arguments' properties may not be accessed on strict mode functions or the arguments objects for calls to them
    at Function.invokeGetter (<anonymous>:3:28)
    at events.<computed>.handler (https://example.com/modules/contrib/conditional_fields/js/conditional_fields.js?sqx68s:197:13)
    at HTMLDocument.dispatch (https://example.com/core/assets/vendor/jquery/jquery.min.js?v=3.7.1:2:40035)
    at v.handle (https://example.com/core/assets/vendor/jquery/jquery.min.js?v=3.7.1:2:38006)
    at Object.trigger (https://example.com/core/assets/vendor/jquery/jquery.min.js?v=3.7.1:2:70124)
    at HTMLDivElement.<anonymous> (https://example.com/core/assets/vendor/jquery/jquery.min.js?v=3.7.1:2:70726)
    at Function.each (https://example.com/core/assets/vendor/jquery/jquery.min.js?v=3.7.1:2:3129)
    at ce.fn.init.each (https://example.com/core/assets/vendor/jquery/jquery.min.js?v=3.7.1:2:1594)
    at ce.fn.init.trigger (https://example.com/core/assets/vendor/jquery/jquery.min.js?v=3.7.1:2:70701)
    at states.Dependent.reevaluate (https://example.com/core/misc/states.js?v=10.3.10:424:22)
🇺🇸United States weekbeforenext Asheville, NC

This seems to be working for me again locally on 4.0.0-alpha6 if I apply the Drupal core patch from https://www.drupal.org/project/drupal/issues/3468860 🐛 Form field #states not woring with entity reference fields Active and revert conditional_fields.js to the alpha5 version. The remaining issues with the javascript can be followed in this issue: https://www.drupal.org/project/conditional_fields/issues/3503448 🐛 Interdependent field conditions not working: "Undefined" value set Active .

🇺🇸United States weekbeforenext Asheville, NC

If anyone needs a patch that can apply to 2.1.0 you can use this file. The MR patches don't apply because the packaging info is in the duration_field.info.yml file. This is a bandaid fix.

🇺🇸United States weekbeforenext Asheville, NC

If anyone needs a patch for 1.0,0-beta6, you can use this one. The other patches don't apply to that version because of the d.o packaging info added to the multivalue_form_element.info.yml file.

🇺🇸United States weekbeforenext Asheville, NC

I resolved merge conflicts in the MR, but the functionality isn't working as expected on my site. I will be circling back to this.

🇺🇸United States weekbeforenext Asheville, NC

weekbeforenext changed the visibility of the branch conditional_fields-3063335-3063335-4 to hidden.

🇺🇸United States weekbeforenext Asheville, NC

weekbeforenext changed the visibility of the branch 3063335 to hidden.

🇺🇸United States weekbeforenext Asheville, NC

Invoice #1042 sent.
Payment transferred, Transaction ID: 252961279
The invoice has been paid.

🇺🇸United States weekbeforenext Asheville, NC

Updated the branch for Merge Request 1 to fix merge conflicts upstream. Looks like I screwed up my git config for Drupal.org to find my account.

🇺🇸United States weekbeforenext Asheville, NC

Moved this issue to the MOSA project.

🇺🇸United States weekbeforenext Asheville, NC

Digging a little more into the code from #3469550 I'm understanding that I can revoke the "Create translations", "Delete translations", and "Edit translations" and leave the "Manage translations for any entity that the user can edit" permission enabled to get the functionality to work as I expect it to...

My bad. Closing this, but I hope it helps others that run across this issue.

🇺🇸United States weekbeforenext Asheville, NC

The tests that are failing are tests I added. I will try to circle back to this on my own time.

🇺🇸United States weekbeforenext Asheville, NC

New config to indicate whether the site allows mixed-case paths or lowercase only. Based on that indicator, we ensure redirects aren't created that cause redirect loops. I think a similar functionality was available in the Drupal 7 version, so I went this route.

I don't know the history of not including it for D8+, so happy to pivot to another path that protects us against redirect loops.

Production build 0.71.5 2024