Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Could you please add the routing and UI to D8 that was in D7?
Comment | File | Size | Author |
---|---|---|---|
#12 | interdiff-2845772-10-12.txt | 439 bytes | abhisekmazumdar |
#12 | 2845772-12.patch | 8.58 KB | abhisekmazumdar |
#10 | interdiff-2845772-8-10.txt | 2.88 KB | abhisekmazumdar |
#10 | 2845772-10.patch | 8.58 KB | abhisekmazumdar |
#8 | interdiff-2845772-6-8.txt | 4.91 KB | abhisekmazumdar |
Comments
Comment #2
dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer commentedBumping this up to Major as this really is a major feature missing from the module.
Comment #3
dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer commentedThis is actually critical as this is blocking the stable release.
Comment #4
abhisekmazumdarComment #5
abhisekmazumdarThis patch will add the following changes:
What's Not working yet:
Note: Apply the patch on the simplytest sandbox environment gives an error.
Comment #6
abhisekmazumdarThings changed in this patch:
Comment #7
sumitmadan CreditAttribution: sumitmadan at QED42 commentedUse $form_state to fetch the selected values.
"dsm" depends on devel module.
Use "_" hyphen instead of "-" underscore for name of form elements. Drupal will convert the hyphen to underscore while generating html.
Instead of setting and unsetting, use a condition inside foreach.
Comment #8
abhisekmazumdarThanks, @sumitmadan.
The suggested changes are done in this new patch.
Comment #9
dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer commentedThanks for working on this @abhisekmazumdar.
This was long overdue and much needed patch. Really appreciate your work on this and this would really help us to finally make a stable release for this module.
Also, really good work on patch:
Nice catch!
Thanks for this.
Suggestions for few changes:
I think this was added by mistake.
.DS_Store is MacOS-specific folder-settings file and is not needed here.
Let's make this 'administer content'
And this as 'administer users'
Typo: selete_node_type.
We can simply call it node_type
Typo here as well
Comment #10
abhisekmazumdarThank You @dipakmdhrm for your suggestion and appreciation.
Comment #11
dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer commentedYikes!!!!
Just realizing that 'administer content' is actually not a valid permission. It's 'administer nodes'.
Totally my fault. Apologies for misleading you in my last comment.
Can you please fix the patch?
Apologies again.
Comment #12
abhisekmazumdarI too missed crossing check it. It's my fault too. I have updated the new patch with the required changes.
Thank You.
Comment #13
dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer commentedComment #15
dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer commentedThank you @abhisekmazumdar for your work on this.
Really good job and this would help us to finally make a stable release for this module.
Comment #17
abhisekmazumdar