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.
When you add or update group, you always get the same message "@type %title has been created."
Comment | File | Size | Author |
---|---|---|---|
#13 | group-fix-delete-success-message-2869019-13.patch | 827 bytes | vasyl.kletsko |
#4 | group-fix-delete-success-message-2869019-4.patch | 825 bytes | vasyl.kletsko |
#5 | group-fix-success-message-2869019-1.patch | 699 bytes | vasyl.kletsko |
#6 | Create Group.png | 31.08 KB | riddhi.addweb |
#6 | Update Group.png | 32.27 KB | riddhi.addweb |
Comments
Comment #2
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedComment #3
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedComment #4
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedAlso, I created the patch that adds success message after a group was deleted.
Comment #5
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedComment #6
riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commentedThanks B@zel for patch, I have applied your patch which is mention in #5 using simplytest.me and issue regarding success message for update group has been fixed. PFA Screenshots.
Comment #7
kristiaanvandeneyndeLooks good, except for the %operation bit. This would make those verbs untranslatable. Will combine the patches and clean them up.
Comment #8
kristiaanvandeneyndePlease try this for creation, updating and deletion.
Comment #9
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedI tested all those actions, works good.
Comment #10
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedComment #12
kristiaanvandeneyndeGreat, thanks all!
Comment #13
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedI improved your code by removing repeated calling of the "t()" method.
Comment #14
vasyl.kletsko CreditAttribution: vasyl.kletsko commentedComment #15
kristiaanvandeneyndeYou can't do that because then the translation parser for LDO doesn't pick up the strings. All t() calls should have a literal string in them instead of a variable.