Hi,
I have installed the following patch :
node/1570902
node/1583722
files/save_emails-1572036-2.patch

I use the "Per node password" configuration whith send mail & send now option

When creating the node, if the password field is empty, I have a drupal message, To protect this page, please enter a password. ....
But When I update a node, the password is not required.( it is present in the database ).
If I click "send immediately the mail", the token [node:password] is empty.

In node_node_update (protected_node.module) there's is : // Get previous emails from database, but I don't see Get previous password from database ?
Regards,

CommentFileSizeAuthor
#2 fix_token-1758996-2.patch3.09 KBizus
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zorax’s picture

I have test to add the line below in the mail.inc, just after the drupal_mail request.
drupal_set_message('password : '.$node->protected_node_passwd );

The variable is always OK, so I wonder if it's not a token problem ?

izus’s picture

Issue summary: View changes
FileSize
3.09 KB

i think both the issue is old and the token problem was not fixed for all parts of code.
the next patch will just fix remaining ald token issues. i will merge it in few minuets.
concerning the password in the email, i tested and it works with this patch.
In the cofiguration page, the token to use is : [node:password] (it is ther by default)
in the node edit form, you should reenter the password to be sent in clear, otherwise no password will be sent because we ignore it as it is not stored clear anywhere !

izus’s picture

Status: Active » Fixed

merged now.
thanks

Status: Fixed » Closed (fixed)

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