Bug 10710: (follow-up) perltidy new code
authorGalen Charlton <gmc@esilibrary.com>
Wed, 14 Aug 2013 16:37:49 +0000 (16:37 +0000)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 28 Aug 2013 14:52:43 +0000 (14:52 +0000)
Also, I prefer the use of parentheses for function
calls, even when no arguments are being passed.
One missed comma turns

is(GetOfflineOperation, ...)

into

is(GetOfflineOperation ...)

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
t/db_dependent/Circulation_OfflineOperation.t

index 46ecb3b..2f5edf8 100644 (file)
@@ -71,7 +71,7 @@ is_deeply(
     {
         operationid => $offline_id,
         userid      => 'User1',
-        branchcode  =>  $samplebranch1->{branchcode},
+        branchcode  => $samplebranch1->{branchcode},
         timestamp   => "0000-00-00 00:00:00",
         action      => 'Action1',
         barcode     => 'CODE',
@@ -80,16 +80,20 @@ is_deeply(
     },
     "GetOffline returns offlineoperation's informations"
 );
-is(GetOfflineOperation,undef,'GetOfflineOperation without parameters returns undef');
-is(GetOfflineOperation(-1),undef,'GetOfflineOperation with wrong parameters returns undef');
+is( GetOfflineOperation(), undef,
+    'GetOfflineOperation without parameters returns undef' );
+is( GetOfflineOperation(-1), undef,
+    'GetOfflineOperation with wrong parameters returns undef' );
 
 #Test GetOfflineOperations
 #TODO later: test GetOfflineOperations
 # Actually we cannot mock C4::Context->userenv in unit tests
 
 #Test DeleteOfflineOperation
-is (DeleteOfflineOperation($offline_id),'Deleted.','Offlineoperation has been deleted');
-#is (DeleteOfflineOperation, undef, 'DeleteOfflineOperation without id returns undef');
+is( DeleteOfflineOperation($offline_id),
+    'Deleted.', 'Offlineoperation has been deleted' );
+
+#is (DeleteOfflineOperation(), undef, 'DeleteOfflineOperation without id returns undef');
 #is (DeleteOfflineOperation(-1),undef, 'DeleteOfflineOperation with a wrong id returns undef');#FIXME
 
 #End transaction