http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5436

Paul Poulain <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ian.walls@bywatersolutions.
                   |                            |com, [email protected]

--- Comment #18 from Paul Poulain <[email protected]> 2011-08-03 
15:15:49 UTC ---
QA comment

* works as announced. beware the updatedatabase is in an atomicupdate file,
must be added in updatedatabase before applying to master

* there are a few reindenting or minor & unnecessary syntax changes, like: 
-    my ($warning_year, $warning_month, $warning_day) = split /-/,
$borrower->{'dateexpiry'};
-    my (  $enrol_year,   $enrol_month,   $enrol_day) = split /-/,
$borrower->{'dateenrolled'};
+    my ($warning_year, $warning_month, $warning_day) = split (/-/,
$borrower->{'dateexpiry'});
+    my (  $enrol_year,   $enrol_month,   $enrol_day) = split (/-/,
$borrower->{'dateenrolled'});
but nothing worth a failed QA

* there are some properties added to staff-global.css, not sure they are all
relevant (cartmenulink sounds strange. collapseaddress is used & OK). I suggest
asking owen

OWEN = I mark it "passed QA", if you notice something strange on staff-global,
please object

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to