From daa3524d4f512f2bf671c6b07c606ba3eec1fefc Mon Sep 17 00:00:00 2001
From: Joshua Goodman <jgoodman1990@gmail.com>
Date: Tue, 25 Feb 2014 05:40:17 +0000
Subject: updated ui to use a dropdown of existing reason types instead of text
 box

---
 httemplate/edit/cust_refund.cgi | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'httemplate')

diff --git a/httemplate/edit/cust_refund.cgi b/httemplate/edit/cust_refund.cgi
index df42e63ae..9f7ac8dee 100755
--- a/httemplate/edit/cust_refund.cgi
+++ b/httemplate/edit/cust_refund.cgi
@@ -106,14 +106,17 @@
     <INPUT TYPE="hidden" NAME="payinfo" VALUE="">
 % }
 
-  <TR>
-    <TD ALIGN="right">Reason</TD>
-    <TD BGCOLOR="#ffffff"><INPUT TYPE="text" NAME="reason" VALUE="<% $reason %>"></TD>
-  </TR>
+<& /elements/tr-select-reason.html,
+              'field'          => 'reasonnum',
+              'reason_class'   => 'F',
+              'control_button' => "document.getElementById('confirm_refund_button')",
+              'cgi'            => $cgi,
+&>
+
 </TABLE>
 
 <BR>
-<INPUT TYPE="submit" NAME="submit" VALUE="Post refund">
+<INPUT TYPE="submit" ID="confirm_refund_button" VALUE="<% mt('Post refund') |h %>" DISABLED>
 
 </FORM>
 
-- 
cgit v1.2.1


From afe30fc7315196cc4ae6da7606995cb8388d1a32 Mon Sep 17 00:00:00 2001
From: Joshua Goodman <jgoodman1990@gmail.com>
Date: Tue, 25 Feb 2014 05:40:44 +0000
Subject: added refund reasons to main menu

---
 httemplate/elements/menu.html | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'httemplate')

diff --git a/httemplate/elements/menu.html b/httemplate/elements/menu.html
index cd4fb39ec..4ba7b2f00 100644
--- a/httemplate/elements/menu.html
+++ b/httemplate/elements/menu.html
@@ -656,6 +656,10 @@ if ( $curuser->access_right('Configuration') ) {
   $config_billing{'separator4'} = ''; #its a separator!
   $config_billing{'Credit reasons'}  = [ $fsurl.'browse/reason.html?class=R', 'Credit reasons explain why a credit was issued.' ];
   $config_billing{'Credit reason types'}  = [ $fsurl.'browse/reason_type.html?class=R', 'Credit reason types define groups of reasons.' ];
+  
+  $config_billing{'separator5'} = ''; #its a separator!
+  $config_billing{'Refund reasons'}  = [ $fsurl.'browse/reason.html?class=F', 'Refund reasons explain why a refund was issued.' ];
+  $config_billing{'Refund reason types'}  = [ $fsurl.'browse/reason_type.html?class=F', 'Refund reason types define groups of reasons.' ];
 }
 
 #XXX also to be unified
-- 
cgit v1.2.1


From 43839f03bf937eb086c7f2a55c059a606571ab77 Mon Sep 17 00:00:00 2001
From: Joshua Goodman <jgoodman1990@gmail.com>
Date: Tue, 25 Feb 2014 05:45:23 +0000
Subject: added refund reason_type class

---
 httemplate/elements/tr-select-reason.html | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

(limited to 'httemplate')

diff --git a/httemplate/elements/tr-select-reason.html b/httemplate/elements/tr-select-reason.html
index 9a670a26b..b7a715b42 100755
--- a/httemplate/elements/tr-select-reason.html
+++ b/httemplate/elements/tr-select-reason.html
@@ -6,8 +6,8 @@ Example:
 
     #required 
     'field'         => 'reasonnum',
-    'reason_class'  => 'C', # currently 'C', 'R',  or 'S'
-                           # for cancel, credit, or suspend
+    'reason_class'  => 'C', # currently 'C', 'R', 'F',  or 'S'
+                           # for cancel, credit, refund, or suspend
 
     #recommended
     'cgi' => $cgi, #easiest way for things to be properly "sticky" on errors
@@ -161,6 +161,8 @@ if ($class eq 'C') {
   $add_access_right = 'Add on-the-fly suspend reason';
 } elsif ($class eq 'R') {
   $add_access_right = 'Add on-the-fly credit reason';
+} elsif ($class eq 'F') {
+  $add_access_right = 'Add on-the-fly refund reason';
 } else {
   die "illegal class: $class";
 }
-- 
cgit v1.2.1