[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Koha-cvs] CVS: koha/circ circulation.pl,1.65,1.66
From: |
Owen Leonard |
Subject: |
[Koha-cvs] CVS: koha/circ circulation.pl,1.65,1.66 |
Date: |
Mon, 26 Jan 2004 11:32:39 -0800 |
Update of /cvsroot/koha/koha/circ
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv1295/circ
Modified Files:
circulation.pl
Log Message:
Added variables available to template: streetaddress, city, phone, cardnumber.
Added FIXME comments to sections where HTML should be moved to template
Index: circulation.pl
===================================================================
RCS file: /cvsroot/koha/koha/circ/circulation.pl,v
retrieving revision 1.65
retrieving revision 1.66
diff -C2 -r1.65 -r1.66
*** circulation.pl 21 Jan 2004 20:49:54 -0000 1.65
--- circulation.pl 26 Jan 2004 19:32:36 -0000 1.66
***************
*** 172,175 ****
--- 172,179 ----
# Barcode entry box, with hidden inputs attached....
+
+ # FIXME - How can we move this HTML into the template? Can we create
+ # arrays of the months, dates, etc and use <TMPL_LOOP> in the template to
+ # output the data that's getting built here?
my $counter = 1;
my $dayoptions = '';
***************
*** 231,234 ****
--- 235,241 ----
$dd=format_date($dd);
$datedue=~s/-//g;
+ # FIXME - Instead of declaring the font color here, can we set a variable
+ # that says 'overdue'? Then the template can check for it: <TMPL_IF
+ # NAME="overdue"><font color="red"></TMPL_IF>
if ($datedue < $todaysdate) {
$dd="<font color=red>$dd</font>\n";
***************
*** 259,262 ****
--- 266,270 ----
$datedue=~s/-//g;
if ($datedue < $todaysdate) {
+ # FIXME - See line 233 above regarding overdues
$dd="<font color=red>$dd</font>\n";
}
***************
*** 307,310 ****
--- 315,322 ----
surname => $borrower->{'surname'},
categorycode => $borrower->{'categorycode'},
+ streetaddress => $borrower->{'streetaddress'},
+ city => $borrower->{'city'},
+ phone => $borrower->{'phone'},
+ cardnumber => $borrower->{'cardnumber'},
question => $question,
barcode => $barcode,
***************
*** 355,358 ****
--- 367,377 ----
$env{'datedue'}='';
} else {
+
+ # FIXME - Can we set two flags here, one that says 'invalidduedate', so that
+ # the template can check for it, and then one for a particular message?
+ # Ex: <TMPL_IF NAME="invalidduedate"> <TMPL_IF NAME="daysinFeb">
+ # Invalid Due Date Specified. Book was not issued. Never that many days
+ # in February! </TMPL_IF> </TMPL_IF>
+
if (($year eq 0) || ($month eq 0) || ($year eq 0)) {
$invalidduedate="Invalid Due Date Specified. Book was not
issued.<p>\n";
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Koha-cvs] CVS: koha/circ circulation.pl,1.65,1.66,
Owen Leonard <=