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.
Problem/Motivation
There is no View link in the local tasks of the contact form. When using modules like clone, devel and entity_usage, it will add new local task links to each contact form. However, those links cannot be easily found unless you know the exact URL for those new links which are not easily known either.
If we have View link in the local tasks of the contact form, we can see all those extra links.
Steps to reproduce
- Install the contact, and devel modules in a new Drupal website
- Create a contact form and go to the contact form page
- the 'Devel' link cannot be found in the contact form page
Proposed resolution
Add a task in the contact module link yml file
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#15 | 3173756-9.patch | 1.59 KB | paulocs |
#8 | 3173756-test-only-9.patch | 1.02 KB | paulocs |
#8 | 3173756-9.patch | 1.59 KB | paulocs |
#6 | Screenshot from 2020-09-29 15-11-05.png | 43.82 KB | paulocs |
#5 | interdiff_2_5.txt | 598 bytes | anmolgoyal74 |
Comments
Comment #2
eric.chenchao CreditAttribution: eric.chenchao commentedComment #3
eric.chenchao CreditAttribution: eric.chenchao commentedComment #4
cilefen CreditAttribution: cilefen commentedComment #5
anmolgoyal74 CreditAttribution: anmolgoyal74 at OpenSense Labs for DrupalFit commentedComment #6
paulocsPatch #5 looks good. The "View" tab is properly created.
RTBC. I attached screen shot to prove it.
Do we need test coverage for this type of issue? I'm not sure.
Comment #7
cilefen CreditAttribution: cilefen commentedThat is documented.
Comment #8
paulocsOkay, so I created test for it.
I attached the test only patch and the complete patch.
Cheers, Paulo.
Comment #10
paulocsComment #12
eric.chenchao CreditAttribution: eric.chenchao commentedLooks good to me
Comment #13
paulocsMoving to RTBC per comment #12.
Comment #14
quietone CreditAttribution: quietone as a volunteer commentedThis is retesting the fail patch instead of the patch for commit. Can someone re-upload the 'success' patch so that is the one tested? Setting to NW for that, sorry.
Comment #15
paulocsRe-uploading patch #9 and moving back to RTBC!
Comment #16
NitinLama CreditAttribution: NitinLama as a volunteer and at OpenSense Labs commentedPatch #15 looks good.
Comment #18
catchCommitted d02178b and pushed to 9.2.x. Thanks!