No, I think there is no problem with the original code.

I did these changes because I would like to improve a little bit its self-documentation. I mean it explicitly show the addActionList() function called if there is not quote defined at all or defined some and this function doesn't call if no quote specified. I try avoid to force anyone who has no experience with this code to examine the code of addActionList() to see it doesn't do anything when the quote points to empty string. 

However If you want the original code back then it is ok.