An often-desired piece of functionality - support printing a list of all form elements which had an error in the message at the top. Perhaps with a !titles token?

Patch to follow.

CommentFileSizeAuthor
#5 ife-1826268-05.patch4.17 KBjames.williams
#4 ife-1826268-04.patch4.67 KBjamsilver
#2 ife-1826268-01-after-1826242-02.patch4.47 KBjamsilver
#1 ife-1826268-01.patch4.43 KBjamsilver
#1 ife-1826268-01-after-1826242-01.patch4.61 KBjamsilver
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jamsilver’s picture

jamsilver CreditAttribution: jamsilver commented
Assigned: jamsilver » Unassigned
Status: Needs work » Needs review
FileSize
ife-1826268-01-after-1826242-01.patch4.61 KB
ife-1826268-01.patch4.43 KB

Patch attached.

The first should apply directly to ife, the 2nd patch applies after applying the patch found here: http://drupal.org/files/ife-1826242-01.patch

jamsilver’s picture

jamsilver CreditAttribution: jamsilver commented
FileSize
ife-1826268-01-after-1826242-02.patch4.47 KB

One that works..

stijndm’s picture

stijndm CreditAttribution: stijndm commented
Status: Needs review » Needs work

Hey jamsilver,

the patch failed against the latest development version. Could you try rerolling the path?

Thx!

jamsilver’s picture

jamsilver CreditAttribution: jamsilver commented
Status: Needs work » Needs review
FileSize
ife-1826268-04.patch4.67 KB

Re-rolled with a couple of tweaks against 2.0-alpha2.

james.williams’s picture

james.williams CreditAttribution: james.williams commented
Version: 7.x-2.0-alpha1 » 7.x-2.0-alpha2
FileSize
ife-1826268-05.patch4.17 KB

This patch is exactly the same as in #4 above, but ensures that !titles is not shown in the resulting message, even if the titles themselves could not be determined.