I wanted to give users more information about their role applications.
So I've replaced the "display_approved" radio button with a set of checkboxes which lets the admin choose to display each of the states of the role applications, and in addition, the roles that the user is already granted and for which there's no application in the database.

Also removed some calls to t() following warning messages like these:

* In format_plural(), the singular and plural strings should be literal strings. There should be no variables, concatenation, constants or even a t() call there. At format_plural(count($received),t('Your application was received for the following role:'),t('Your applications were received for the following roles:')) in apply_for_role.module on line 544.
* Invalid menu 'title' definition found in apply_for_role_menu(). Title and description keys of the menu array should be literal strings. In apply_for_role.module on line 60.
* Invalid menu 'description' definition found in apply_for_role_menu(). Title and description keys of the menu array should be literal strings. In apply_for_role.module on line 61.

And a Hebrew translation for the user displayed strings is attached.

Hope you will find this useful.

Comments

geneticdrift’s picture

StatusFileSize
new14.61 KB
new10.58 KB

Where's the attachments gone?
Trying again...

jnicola’s picture

Version: » 7.x-2.x-dev
Issue summary: View changes
Status: Needs review » Closed (fixed)

In the 7.x this is straight forward and already done. We have approved and denied, and if there is nothing it speaks for itself as being pending.