🇨🇷Costa Rica @MaxMendez

A new version that removed a cron warning related with feeds book target plugin.

@profwebsait did you use this patch in tandent with https://www.drupal.org/files/issues/2021-12-16/feeds_para_mapper-error-g... →

This is a simple very patch, i've tested and wotk fine. Thanks for your time.

Thanks for me on Drupal 10.2.3 and 8.x-1.1

The patch #4 fixed my problem.

Please try to use tis steps to reproduce:

• Set a layout to default view mode of a content type and display some field.
• Active a new mode, it will be inherit default view mode settings.
• Go to the new view mode activated.
• Set --None-- as layout and try to save.

Today I faced with the same requirement, i've reviewed #12 comment code and created an tested in my case of use and seems to work correctly.

I think this code requiere to be improved or reviewed by one with more experience with feeds and this module code and behavior.

Today I faced this error on Drupal 10.1.6 (PHP 8.1.25) and Views fieldsets 4.0.0:

Error: Call to a member function get() on null en Drupal\views_ui\Form\Ajax\Rearrange->submitForm() (linea 157 de /var/www/tutorial.avatarsys.app/web/core/modules/views_ui/src/Form/Ajax/Rearrange.php).

And patch #3 works for me. Thanks!

Hi @darvanen, as you can see the change of status was a month later of the path, but no one cared about this module for month, but now when time is short it is easy come and talk about the work of others.

Why don´t drop D8 support?, the original didn't and was my mistake did not drop that moment. Be free to improve and create a pr or patch to help to get a new release soon.

Why the desition of accessCheck(FALSE)?
Well, really thanks @darvanen of time of review and testing in how the the module work and conclude what makes the most sense.

Thanks for comments, i'll do that.

MaxMendez → created an issue.

Hi @admirlju, i've been testing it and seems to be complete and working properly.

It can be review.

Hi @robphillips , thanks for your answer and time, i'll work around your suggestion.

Tested patch #8 and works good in my case.

Thanks mates for your work.

Fixed a little error on logic.

Here a first approach

MaxMendez → created an issue.

Hi @MukhtarM

My first observation was wrong because the once library returns elements, but as native selector not jQuery compatible, so .attr is not available o line 21.

I applied MR 34 using D10.1.2, PHP8.2 and 5.0.0-rc1, add a block using the filter in the process and this is the result:

Rethinking in the problem i simplified the patch to fix the problem and maintaining the original logic.

Tested the pr and seems that requires changes on the original logic since new once library does not return the element selected.

Here my initial approach, in the future i think jquery dependency should be remove using vanilla js, but it could be in other issue.

MaxMendez → created an issue.

This problem persist on 5.5.12 and 5.5.x-dev.

Seems to be only in the compile files.

Hi @AdamPS,

I've tested it in my dev commerce environment and seems that the problem is fixed. Thanks for your work.

+1 for RTBC on #2

Hi,

Tested patch an it works fine, thanks.

Here a first aproach.

- Changed core_version_requirement, removed 8 compatibility and updated minimum version of 9
- Updated library definition to load core.once library.
- Updated js to use core.once.

Tested under Drupal 10.1.0

Tested patch from #33 on D 10.1 and works perfectly.

Thanks for your effort and time.

Removes an unnecessary file

Sorry i did not include the new schema file. Here a new patch.

Here my approach,

1- Removed override of ckeditor 5 schema, change schema file name using name of the module and remove all default ckeditor5 records.
2- Updated Font Plugin to 38.0.1 same version uses by core module ( https://www.drupal.org/project/drupal/releases/10.1.0#frontend-deps → ).

Please test in your environments if all is working good, in mine seems to work perfectly.

I've tested MR 25 too and it fixes the last D10 issue

Same fatal error.

In my case, i'm using "drupal/cshs": "^4.0" an the patch work without problems. Did you use the stable package or dev?

Tested merge request #13 and works perfectly. Thanks @hanoii for your work!.

Hi,

I faced with this warning today, in my case the problem was trigger by the options 'Translate "Then" output' and 'Translate "Or" output', i disabled both options and disappear, in my case i does not neet translation of the values and this could be a clue to solve the root problem.

Hi @chucksimply,

The patch requires the dev version.

Under composer require:

    "require": {
         "drupal/views_fieldsets": "3.x-dev@dev",

And under drupal-lenient:

        "drupal-lenient": {
            "allowed-list": [
                "drupal/pdf",
                "drupal/views_fieldsets"
            ]
        }

Note: the last one if you use Lenient.

Another temporal option is the plugin https://packagist.org/packages/mglaman/composer-drupal-lenient

Fixed and error in the while condition.

I faced this issue in my account that have more that one page of zones, reviewing the logic Zone->list Zones seems that creates an infinite loop.

Here my approach to fixed it.

Here the global patch updated.

Updated patch for latest version of branch 8.x-3.x-dev.

Updated core_version_requirement, removed D8 compatibility and added D10

Here my approach to fixed the problem.

MaxMendez → created an issue.

Sorry, my mistake, it is fixed but has been released core versión with this patch.

I'm using 9.5.9 and the problems seems to be solved for the admin user (user 1), but the problems seems to persist on user with low level of permissions.

ntrolgroup.js:39 Uncaught TypeError: t.widget is not a function
    at controlgroup.js:39:10
    at controlgroup.js:32:3
    at controlgroup.js:19:1
(anónimo) @ controlgroup.js:39
(anónimo) @ controlgroup.js:32
(anónimo) @ controlgroup.js:19
form-reset-mixin.js:34 Uncaught TypeError: Cannot set properties of undefined (setting 'formResetMixin')
    at form-reset-mixin.js:34:13
    at form-reset-mixin.js:29:3
    at form-reset-mixin.js:15:1
(anónimo) @ form-reset-mixin.js:34
(anónimo) @ form-reset-mixin.js:29
(anónimo) @ form-reset-mixin.js:15
mouse.js:40 Uncaught TypeError: e.widget is not a function
    at mouse.js:40:10
    at mouse.js:30:3
    at mouse.js:15:1
(anónimo) @ mouse.js:40
(anónimo) @ mouse.js:30
(anónimo) @ mouse.js:15
checkboxradio.js:40 Uncaught TypeError: Cannot read properties of undefined (reading 'formResetMixin')
    at checkboxradio.js:40:38
    at checkboxradio.js:35:3
    at checkboxradio.js:20:1
(anónimo) @ checkboxradio.js:40
(anónimo) @ checkboxradio.js:35
(anónimo) @ checkboxradio.js:20
draggable.js:42 Uncaught TypeError: Cannot read properties of undefined (reading 'mouse')
    at draggable.js:42:32
    at draggable.js:37:3
    at draggable.js:17:1
(anónimo) @ draggable.js:42
(anónimo) @ draggable.js:37
(anónimo) @ draggable.js:17
resizable.js:41 Uncaught TypeError: Cannot read properties of undefined (reading 'mouse')
    at resizable.js:41:32
    at resizable.js:36:3
    at resizable.js:19:1
(anónimo) @ resizable.js:41
(anónimo) @ resizable.js:36
(anónimo) @ resizable.js:19
button.js:44 Uncaught TypeError: t.widget is not a function
    at button.js:44:3
    at button.js:39:3
    at button.js:19:1
(anónimo) @ button.js:44
(anónimo) @ button.js:39
(anónimo) @ button.js:19
dialog.js:49 Uncaught TypeError: i.widget is not a function
    at dialog.js:49:3
    at dialog.js:44:3
    at dialog.js:19:1
(anónimo) @ dialog.js:49
(anónimo) @ dialog.js:44
(anónimo) @ dialog.js:19
widget.js:104 Uncaught TypeError: i is not a constructor
    at t.widget (widget.js:104:18)
    at dialog.jquery-ui.js?v=9.5.9:10:5
    at dialog.jquery-ui.js?v=9.5.9:61:3
t.widget @ widget.js:104
(anónimo) @ dialog.jquery-ui.js?v=9.5.9:10
(anónimo) @ dialog.jquery-ui.js?v=9.5.9:61
widget.js:81 Uncaught RangeError: Maximum call stack size exceeded
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
    at new t.<computed>.<computed> (widget.js:81:11)
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
t.<computed>.<computed> @ widget.js:81
drupal.js?v=9.5.9:14 Uncaught TypeError: Cannot read properties of null (reading 'setAttribute')
    at custom-entity-forms.js?v=1.x:7:54
    at NodeList.forEach (<anonymous>)
    at Object.attach (custom-entity-forms.js?v=1.x:4:52)
    at drupal.js?v=9.5.9:24:24
    at Array.forEach (<anonymous>)
    at Drupal.attachBehaviors (drupal.js?v=9.5.9:21:34)
    at drupal.init.js?v=9.5.9:24:12
    at HTMLDocument.listener (drupal.init.js?v=9.5.9:14:7)
(anónimo) @ custom-entity-forms.js?v=1.x:7
attach @ custom-entity-forms.js?v=1.x:4
(anónimo) @ drupal.js?v=9.5.9:24
Drupal.attachBehaviors @ drupal.js?v=9.5.9:21
(anónimo) @ drupal.init.js?v=9.5.9:24
listener @ drupal.init.js?v=9.5.9:14
setTimeout (asíncrono)
Drupal.throwError @ drupal.js?v=9.5.9:13
(anónimo) @ drupal.js?v=9.5.9:26
Drupal.attachBehaviors @ drupal.js?v=9.5.9:21
(anónimo) @ drupal.init.js?v=9.5.9:24
listener @ drupal.init.js?v=9.5.9:14
drupal.js?v=9.5.9:14 Uncaught TypeError: Cannot read properties of null (reading 'setAttribute')
    at custom-entity-forms.js?v=1.x:7:54
    at NodeList.forEach (<anonymous>)
    at Object.attach (custom-entity-forms.js?v=1.x:4:52)
    at drupal.js?v=9.5.9:24:24
    at Array.forEach (<anonymous>)
    at Drupal.attachBehaviors (drupal.js?v=9.5.9:21:34)
    at big_pipe.js?v=9.5.9:43:14
    at Array.forEach (<anonymous>)
    at bigPipeProcessDocument (big_pipe.js?v=9.5.9:42:55)
    at big_pipe.js?v=9.5.9:56:12
(anónimo) @ custom-entity-forms.js?v=1.x:7
attach @ custom-entity-forms.js?v=1.x:4
(anónimo) @ drupal.js?v=9.5.9:24
Drupal.attachBehaviors @ drupal.js?v=9.5.9:21
(anónimo) @ big_pipe.js?v=9.5.9:43
bigPipeProcessDocument @ big_pipe.js?v=9.5.9:42
(anónimo) @ big_pipe.js?v=9.5.9:56
setTimeout (asíncrono)
Drupal.throwError @ drupal.js?v=9.5.9:13
(anónimo) @ drupal.js?v=9.5.9:26
Drupal.attachBehaviors @ drupal.js?v=9.5.9:21
(anónimo) @ big_pipe.js?v=9.5.9:43
bigPipeProcessDocument @ big_pipe.js?v=9.5.9:42
(anónimo) @ big_pipe.js?v=9.5.9:56
setTimeout (asíncrono)
bigPipeProcess @ big_pipe.js?v=9.5.9:55
(anónimo) @ big_pipe.js?v=9.5.9:61
(anónimo) @ big_pipe.js?v=9.5.9:68

I want to help maintaining this module, i use it regularly on my site building and it is a important key of it. I really like to turns it compatible with Drupal 10 and future versions.

I use to test it regulary in order to fixed any warning or error, like https://www.drupal.org/project/context_groups/issues/3286758#comment-149... 📌 Automated Drupal 10 compatibility fixes Needs work .

I've not previously received security coverage opt-in permission.

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

Hi @icurk and @RokSiEu,

Sorry for tagged you but some of you can review this patch and release a new version D10 compatible.

Thanks for your time and help.

Confirmed patch #54 works for me on 9.5.7

Thanks, I'll tell you if there are any problem.

Note: dev release seems to be stuck on 2021.

MaxMendez → created an issue.

I've tested the patch and scan by my own and found another compatibility, here my patch to turn the module Drupal 10 compatible.

This patch i so simple, i've tested and seems to work as a charm.

MaxMendez → created an issue.

Thanks @andyg5000 your suggestion fixed my problem with a custom action button.

Hi, there is any plan to merge it and create a new release compatible with D10?