Welcome to the nuBuilder Forums!

Register and log in to access exclusive forums and content available only to registered users.

[Fixed] nuemailer.php fix

Questions related to using nuBuilder Forte.
Post Reply
apmuthu
Posts: 249
Joined: Sun Dec 06, 2020 6:50 am
Location: Chennai, India, Singapore

[Fixed] nuemailer.php fix

Unread post by apmuthu »

Please pull in the fix for nuemailer.php file from my fork of the repo. I maintain the fixes in a separate folder so as to keep track of the upstream repo independently.
Last edited by apmuthu on Sun Feb 07, 2021 9:52 pm, edited 1 time in total.
kev1n
nuBuilder Team
Posts: 4307
Joined: Sun Oct 14, 2018 6:43 pm
Has thanked: 71 times
Been thanked: 447 times
Contact:

Re: nuemailer.php fix

Unread post by kev1n »

Hi,

What exactly was fixed here?
I just noted that

Code: Select all

trim($obj->$key)
is now assigned to a new variable, eliminating a redundany.
apmuthu
Posts: 249
Joined: Sun Dec 06, 2020 6:50 am
Location: Chennai, India, Singapore

Re: nuemailer.php fix

Unread post by apmuthu »

PHP 5.4.27 on XAMPP v1.8.2.5 gave the error that a function cannot be used in a return value in write context and hence the workaround.

Other workarounds suggest to use "!" instead of "empty" as in:
https://stackoverflow.com/questions/153 ... te-context
and
https://mycyberuniverse.com/how-fix-can ... ntext.html
but I found my way easy to do, understand and also avoid redundancy too.
kev1n
nuBuilder Team
Posts: 4307
Joined: Sun Oct 14, 2018 6:43 pm
Has thanked: 71 times
Been thanked: 447 times
Contact:

Re: nuemailer.php fix

Unread post by kev1n »

Got it, thanks for the clarification !
kev1n
nuBuilder Team
Posts: 4307
Joined: Sun Oct 14, 2018 6:43 pm
Has thanked: 71 times
Been thanked: 447 times
Contact:

Re: nuemailer.php fix

Unread post by kev1n »

FYI, your fix will soon be included. Thanks again for that!
apmuthu
Posts: 249
Joined: Sun Dec 06, 2020 6:50 am
Location: Chennai, India, Singapore

Re: nuemailer.php fix

Unread post by apmuthu »

Fixed as of 2021-12-06 by commit in v4.5
Post Reply