It seems that ORDER_COMMENTS_RMA_OPTION is missing from the configuration and thus doesn't update the order status history with the comments from the call in the template file.
Also, for any forum members that are wondering if this works with 1.3.x, I have tested this thoroughly on 1.3.x and if the last 2 blocks of code at the bottom of admin/includes/init_includes/init_ra_config are commented out (after the last messagestack line) it will install without any issue and works identically to 1.5.x (but don't post anything here since this thread is ONLY for 1.5.x). I found some other stuff that I am going to fix right away, mainly broken text email format from using html code in a text email and a couple other layout/format and niggly little things, to rerelease as a new package for all versions.
A few options that I am going to add include the ability to set and check against a return period if the store has a timeframe that returns are allowed (ours is 30 days) with several optional ways to choose when the timeframe starts from simple to more complex. If the return time is exceeded the form will not process and they will get a message instead saying the return period has expired(I am 3/4 of the way through writing this right now). I am also planning on allowing the admin to set a window for how long the authorization is valid with an automatic email sent at an admin configured time to remind the customer that time is running out and an automatic switch to change the order status and notify the admin and customer when the time has expired (we get some customers that never send stuff back or it might be like 3 months after the authorization was sent out). As well as a couple other tweaks to make it more automated, optional feature rich, streamlined, etc. I will create a new module to upload in it's own slot in the plugins and a separate support thread (look for them in this forum in the next week or so, it will just start with this basic module and bugfixes and I will update the module as new features are added) that will be for all versions of zen cart since this thread is rigidly only for 1.5.x even though it works fine in 1.3.x. I would much rather contribute here but I don't use 1.5.x on my production sites.
One last note, if anyone is using the cherry zen template, in includes/templates/your_custom_template/templates/tpl_returns_default you will need to add an extra </div> at the end of line 40 to prevent the layout from being broken on the success page:
echo '<div id="returnSuccess">' . TEXT_SUCCESS . '</div>' . '<div id="returnRequired">'. TEXT_SUCCESS_RMA_REQUIRED . '</div>' . '<div id="returnPolicy">'. TEXT_SUCCESS_RMA_POLICY_BOF . '<a href="' . zen_href_link(FILENAME_SHIPPING, '', 'SSL') . '">' . TEXT_SUCCESS_RMA_POLICY_LINK . '</a>' . TEXT_SUCCESS_RMA_POLICY_EOF . '</div>' . '<div id="returnAddressWrapper">' . '<div id="returnRMA">' . TEXT_SUCCESS_RMA_ID . $order_number . $rma_number . '</div>' . '<div id="returnAddress">' . TEXT_SUCCESS_RMA_RETURN_ADDRESS . '</div>'; if (RETURN_STORE_NAME_ADDRESS_SUCCESS == 'true') { echo '<address>' . nl2br(STORE_NAME_ADDRESS) . '</address>'; } else if (RETURN_STORE_NAME_ADDRESS_SUCCESS == 'false') { echo '<address>' . nl2br(RETURN_STORE_NAME_ADDRESS_DIFF) . '</address>'; } echo '<div id="returnPhone">' . TEXT_SUCCESS_RMA_RETURN_PHONE . '</div>' . '</div>';
becomes this:
echo '<div id="returnSuccess">' . TEXT_SUCCESS . '</div>' . '<div id="returnRequired">'. TEXT_SUCCESS_RMA_REQUIRED . '</div>' . '<div id="returnPolicy">'. TEXT_SUCCESS_RMA_POLICY_BOF . '<a href="' . zen_href_link(FILENAME_SHIPPING, '', 'SSL') . '">' . TEXT_SUCCESS_RMA_POLICY_LINK . '</a>' . TEXT_SUCCESS_RMA_POLICY_EOF . '</div>' . '<div id="returnAddressWrapper">' . '<div id="returnRMA">' . TEXT_SUCCESS_RMA_ID . $order_number . $rma_number . '</div>' . '<div id="returnAddress">' . TEXT_SUCCESS_RMA_RETURN_ADDRESS . '</div>'; if (RETURN_STORE_NAME_ADDRESS_SUCCESS == 'true') { echo '<address>' . nl2br(STORE_NAME_ADDRESS) . '</address>'; } else if (RETURN_STORE_NAME_ADDRESS_SUCCESS == 'false') { echo '<address>' . nl2br(RETURN_STORE_NAME_ADDRESS_DIFF) . '</address>'; } echo '<div id="returnPhone">' . TEXT_SUCCESS_RMA_RETURN_PHONE . '</div>' . '</div></div>';
Thanks again to the authors for sharing the code, it's so much better and more useful than the original!
