Support Forum
Support Forum
Offline
No - there is still an issue remaining @mr-papa - I have missed something. Will dig in this morning
 |
YELLOW
SWORDFISH
|
Offline
Somewhere along the line we seem to have accidentally lost 2 lines of code here. I will go back and see if I can find why but at least we can get you back to working properly first.
This time the file is:
/html-email/admin/sp-html-email-admin-pw-change-save.php
Before the call to the function sp_update_option() near the bottom, add these two extra lines of code:
$option['pw-change-subject'] = sp_filter_save_escape(sp_filter_save_nohtml(trim($_POST['pw-change-subject']))); $option['pw-change-body'] = sp_filter_text_save(trim($_POST['pw-change-body']));
and hopefully, this time, we HAVE nailed it!
|
YELLOW
SWORDFISH
|
Offline
Mr Papa said
that is an interesting error but unrelated... unfortunately, I am on the road without easy access to the codebase to help much... and Andy is likely gone for the day (in UK)...Did you 'use' his code or type it manually? if the former, did you use the view raw code button and copy? otherwise you might have gotten some different characters...
I wrote manually, using notepad. In file there is $pw also at line 312 and 314, have I to delete also in these lines?
Please note: I can't change the text in any way, also if I type "hello world" and save.
Offline
Yellow Swordfish said
Somewhere along the line we seem to have accidentally lost 2 lines of code here. I will go back and see if I can find why but at least we can get you back to working properly first.This time the file is:
/html-email/admin/sp-html-email-admin-pw-change-save.phpBefore the call to the function sp_update_option() near the bottom, add these two extra lines of code:
<br />
$option['pw-change-subject'] = sp_filter_save_escape(sp_filter_save_nohtml(trim($_POST['pw-change-subject']))); $option['pw-change-body'] = sp_filter_text_save(trim($_POST['pw-change-body']));<br />
and hopefully, this time, we HAVE nailed it!
Done, but, sorry, no good news: still can't change any word. As I wrote to Mr Papa, $pw is in lines 312, 314, too. Have to delete?
Offline
Oh dear - I really wasn't having a good day yesterday was I...
It was the two $pw instances at lines 312 and 314 that I meant to ask you to remove. I pasted the wrong function into the post. But that doesn't matter as the function I pasted in doesn't have the $pw so there was nothing to remove. I assume you did not remove anything from the function I stated yesterday but please nopw do remove the $pw from 312 and 314,
I said earlier that the change password email seemed to be working for us but I was mistaken hence this mornings changes. And these are really a separate issue. However, those two extra lines have fixed it in my tests. You did put them just BEFORE the call to sp_update_option() - yes?
|
YELLOW
SWORDFISH
|
Offline
All done, but no way. Could you please send me your full plug-in dir? I could try to upload it by http://ftp....
Offline
In sp-html-email-plugin.php
line 49
add_filter('retrieve_password_title', 'sp_html_email_pw_change_subject');all other lines have numbers. Have I to add something?
All RSS1 Guest(s)