Return Authorization Module (RMA)

[xspresso]

OK, so I put together code to display the success page.

WHAT THE FOLLOWING DOES

using the code from post#579

1.) I add a RMA Request button on the MY ACCOUNT > HISTORY > ORDER INFO page
2.) When clicked it sends that $orders_id to the returns page and populates the Order Number Field
3.) Send the same $orders_id to the returns success page

Now this code recreates and does away with the HTML from @xspresso post#581

Screenshot:
Attachment 12384

First lets add the code to /includes/templates/YOUR_TEMPLATE_NAME/templates/tpl_returns_default.php

find:

<?php if (RETURN_STORE_NAME_ADDRESS == 'true') { ?>
<address><?php echo nl2br(STORE_NAME_ADDRESS); ?></address>
<?php } ?>

<?php
  if (isset($_GET['action']) && ($_GET['action'] == 'success')) {
?>

<br class="clearBoth" />
<div class="mainContent success"><?php echo TEXT_SUCCESS; ?></div>

<?php
  } else {
?>

Replace with:

<?php if (RETURN_STORE_NAME_ADDRESS == 'true' && ($_GET['action'] == 'success')) { ?>
<?php
/**
show nothing
 */
} else if (RETURN_STORE_NAME_ADDRESS == 'true') { ?>
<address><?php echo nl2br(STORE_NAME_ADDRESS); ?></address>
<?php } ?>

<?php
  if (isset($_GET['action']) && ($_GET['action'] == 'success')) {
?>
<br class="clearBoth" />

<div class="mainContent success">
<?php 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 = $_GET['order_id']; echo $order_number . TEXT_SUCCESS_DASH; srand(5); echo(rand(1, 10)); echo(rand(1, 10)) . TEXT_SUCCESS_DASH . date('mdY') . '</div>' . '<div id="returnAddress">' . TEXT_SUCCESS_RMA_RETURN_ADDRESS . '</div>'; if (RETURN_STORE_NAME_ADDRESS == 'true') { echo '<address>' . nl2br(STORE_NAME_ADDRESS) . '</address>'; } echo '<div id="returnPhone">' . TEXT_SUCCESS_RMA_RETURN_PHONE . '</div>' . '</div>'; ?>

<?php
  } else {
?>

then open /includes/languages/english/YOUR_TEMPLATE_NAME/returns.php

and add:

define('TEXT_SUCCESS_RMA_REQUIRED', 'The below RMA# is required for all Returns');
define('TEXT_SUCCESS_RMA_POLICY_BOF', 'You can view our ');
define('TEXT_SUCCESS_RMA_POLICY_LINK', 'Returns Policy');
define('TEXT_SUCCESS_RMA_POLICY_EOF', ' here.');
define('TEXT_SUCCESS_RMA_ID', 'Your RMA# is: ');
define('TEXT_SUCCESS_DASH', '-');
define('TEXT_SUCCESS_RMA_RETURN_ADDRESS', 'Please ship all returns to this address:');
define('TEXT_SUCCESS_RMA_RETURN_PHONE', 'Phone: 1.111.111.1111'); 

and the CSS rules can be added to /includes/templates/YOUR_TEMPLATE_NAME/css/returns.css

div#returnAddressWrapper {border:1px solid #E9E9E9;background:#FFE573;text-align:center;}
div#returnSuccess {font-size:1.2em;padding:5px;color:#606060;}
div#returnAddressWrapper, div#returnRequired, div#returnPolicy, div#returnRMA, div#returnAddress, div#returnPhone {font-size:1.5em;font-weight:bold;padding:5px; }
div#returnRequired, div#returnAddress {color:#FF0000;}

That should be it, Enjoy!

That is very nice work. It is everything I have been trying to do for a week.

[rbarbour]

That is very nice work. It is everything I have been trying to do for a week.

I'm glad we got it sorted and working

[xspresso]

This allows the customer to receive the RMA # in their email.
I also laid-out the email to look better.

    // Prepare extra-info details
    $extra_info = email_collect_extra_info($name, $email_address, $customer_name, $customer_email, $telephone);
    // Prepare Text-only portion of message
	$text_message = OFFICE_FROM . "\t" . $name . "\n<br />" . 
		    OFFICE_EMAIL . "\t" . $email_address . "\n<br />" .
			"Phone Number:" . "\t" .$telephone . "\n<br />" .
		    "Address:" . "\t" . $address . "\n<br />" .
		    "City:" . "\t" . $city . "\n<br />" .
			"State:" . "\t" . $state . "\n<br />" .
		    "Post Code:" . "\t" . $postcode . "\n<br />" .
			"Country:" . "\t" . $country . "\n<br />" .
		    "Order Number:" . "\t" . $order_number . "\n<br />" .
		    "Total Value:" . "\t" . $value . "\n<br />" .
		    "Item Number:" . "\t" . $item_number . "\n<br />"	.
		    "Item(s) Name:" . "\t" . $item_name . "\n<br />"	.
		    "Action Requested:" . "\t" . $action . "\n\n<br />"	.
		     "RMA Number:" . "\t" . $order_number  . "-". date('mdY') . "\n<br />" .                  
			'------------------------------------------------------<br />' .
                   "\n\n<br />Reason:".    "\n\n<br />" .
                   "" . $reason .   "\n<br />" .
                  '------------------------------------------------------<br />' .  
      "" . "\n<br />" . 
      $extra_info['TEXT'];
      $email_text = sprintf(EMAIL_GREET_NONE, $name );
      $email_text .=  "\n <br />";
      $email_text .=  EMAIL_WELCOME;
      $email_text .=  "\n\n" . "Request Date:" . "\t" . date('m/d/Y') . "\n<br />" ;
      $email_text .=  "\n" . "Invoice Number:" . "\t" . $order_number . "<br />" ;
      $email_text .=  "\n" . "Item(s)You Are Returning: (as you entered it)" . "\t" . $item_name . "<br />";
      $email_text .=  "\n" . "RMA Number:" . "\t" . $order_number  . "-". date('mdY') . "\n<br />";
      $email_text .= "\n\n" . EMAIL_TEXT . EMAIL_CONTACT . EMAIL_WARNING;

I have a concern from post # 588. The code I gave you for the generator srand(5); echo(rand(1, 10)); echo(rand(1, 10)) is wrong.
This one is a 3 digit generator. BUT... my problem is: using this will make the "emailed RMA" different from the RMA that comes back in the success page. So maybe just remove it all together.

$digits = 3; 
echo(rand(pow, 10)); 
$digits-1)), pow(10, $digits)-1))

unless you can carry the generated 3 digit number to the email:

$email_text .=  "\n" . "RMA Number:" . "\t" . $3digit_number  . "-". $order_number  . "-". date('mdY') . "\n<br />";

[rbarbour]

Seeing how the random 3 digit number doesn't really serve a purpose anyway, why not just replace it with a value that could help reference the RMA# with the:

A) Original Order Number
B) Customer's ID
C) RMA# Request Date

This way it can be sent to email when form is submitted and passed to the success page - and never regenerating a different RMA#

[DivaVocals]

Seeing how the random 3 digit number doesn't really serve a purpose anyway, why not just replace it with a value that could help reference the RMA# with the:

A) Original Order Number
B) Customer's ID
C) RMA# Request Date

This way it can be sent to email when form is submitted and passed to the success page - and never regenerating a different RMA#

nods in agreement.. these elements will make the number sufficiently unique without the need for a random generated number.. No need to do anything fancy..

[xspresso]

That is good, just drop the random part and now the order# (aka: customer id or invoice no.) and date is more then enough to track the order as an RMA.

Now there is one final thing I believe the RMA needs to do.

Change the database status from complete (or whatever) to RMA Requested.

I think in header_php.php after sending mail, open database and change the status.

[rbarbour]

Maybe in a future release