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

Paul Poulain <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
       Patch Status|Signed Off                  |Failed QA

--- Comment #9 from Paul Poulain <[email protected]> 2011-10-06 
11:55:37 UTC ---
Sorry for being long to QA, but I've to mark "failed QA" because there are many
lines that are unrelated to the goal of the patch.
Those lines should not harm, but I think they should be removed from the patch:

For example :
-    my $bsth =$dbh->prepare("SELECT branchcode,branchname FROM branches"); #
FIXME : use C4::Branch::GetBranches
+    my $bsth =$dbh->prepare("SELECT branchcode,branchname FROM branches ");    
-                # item is withdrawn, lost, damaged, not for loan, reserved or
in transit
+                # item is withdrawn, lost or damaged

-                    
-                    # can place hold on item ?
+
+ # can place hold on item ?

An many:             
+
(empty line added)

-- 
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