Hi, my users won't know what a node is and I'm not even really asking them to enter a password, but rather to verify the state in which they live in order to proceed to the next protected page.

I was wondering if there is anyway this can be added as a feature or if you could tell me which file I can change this in and how.

Thanks!

CommentFileSizeAuthor
#8 protected_node_add_title_node_type_to_password_screen-5.x.patch5.18 KBjrglasgow
#7 protected_node_add_title_node_type_to_password_screen_6.x.patch4.71 KBjrglasgow
#5 protected_node_add_title_node_type_to_password_screen.patch5.25 KBjrglasgow

Comments

nationalwind’s picture

nationalwind commented

I found where to make the change in the files, however it would still be nice to see this added into the user interface for admins.

Also, if it would be possible to allow the instructions/information to be different per protected node (the text above the box with the password field in it) that would be great.

mtolmacs’s picture

mtolmacs commented

I see now that this part of the module has to be re-invented from the ground up. You're not the only one who asks for similar but not exactly this functionality. I want to solve this problem once. Thank you for your patience.

esadot’s picture

esadot commented

Needed the same, and came up with a tiny module:

Module name: my_module

function my_module_form_alter($form_id, &$form) {
  if ($form_id == 'protected_node_enterpassword') {
    $form['protected_node_enterpassword']['#description'] = 'Text here';
    $form['protected_node_enterpassword']['password']['#title'] = 'Text here';
  }
}
jrglasgow’s picture

jrglasgow commented

It would help me if you could change the password form, when entering the password allow the user setting the password to choose to give those who see the password form a title for the page.

If anything change the message from "Protected node - Enter Password" to "Protected {$node_type} - Enter Password" so it might say "Protected Page..." or "Protected Story..." and instead of "Node Password:" change it to "{$node_type} Password:" so you would get "Page Password:" or "Story Password:"

jrglasgow’s picture

jrglasgow commented
StatusFileSize
new protected_node_add_title_node_type_to_password_screen.patch5.25 KB

for my needs in #4 I have created a patch for Drupal 5, I will see if I can find the time today to write a patch for D6.

This patch will do the following:
When creating the protected node the user now has an option to "Show the title to users when requesting the password."

Also when the user views the password page it will instead of talking about the "Node" it will display the node type.

nationalwind’s picture

nationalwind commented

jrglasgow,

I would appreciate this in D6, if you have time! In a previous version, I was able to hard code a change to the title into the module, but somehow that doesn't seem to be working anymore.

Thanks

jrglasgow’s picture

jrglasgow commented
Status: Active » Needs review
StatusFileSize
new protected_node_add_title_node_type_to_password_screen_6.x.patch4.71 KB

try this patch for D6

jrglasgow’s picture

jrglasgow commented
StatusFileSize
new protected_node_add_title_node_type_to_password_screen-5.x.patch5.18 KB

I found a small problem in the D5 patch, here is the replacement.

nationalwind’s picture

nationalwind commented

Thanks. The change in the module worked after all (as expected). Just had to clear my cache like ten times!

mtolmacs’s picture

mtolmacs commented
Status: Needs review » Fixed

I've committed the patch. Thank you!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

sreese’s picture

sreese commented

It doesn't appear as though the Drupal 5.x patch was applied to version 5.x-1.4. Any particular reason why not?