]> git.agnieray.net Git - galette.git/commitdiff
Throw errors
authorJohan Cwiklinski <johan@x-tnd.be>
Sat, 6 Nov 2021 05:57:07 +0000 (06:57 +0100)
committerJohan Cwiklinski <johan@x-tnd.be>
Sun, 7 Nov 2021 05:50:39 +0000 (06:50 +0100)
32 files changed:
galette/lib/Galette/Core/History.php
galette/lib/Galette/Core/Links.php
galette/lib/Galette/Core/Login.php
galette/lib/Galette/Core/MailingHistory.php
galette/lib/Galette/Entity/Adherent.php
galette/lib/Galette/Entity/Contribution.php
galette/lib/Galette/Entity/ContributionsTypes.php
galette/lib/Galette/Entity/DynamicFieldsHandle.php
galette/lib/Galette/Entity/Entitled.php
galette/lib/Galette/Entity/FieldsCategories.php
galette/lib/Galette/Entity/FieldsConfig.php
galette/lib/Galette/Entity/Group.php
galette/lib/Galette/Entity/ImportModel.php
galette/lib/Galette/Entity/ListsConfig.php
galette/lib/Galette/Entity/PaymentType.php
galette/lib/Galette/Entity/PdfAdhesionFormModel.php
galette/lib/Galette/Entity/PdfContribution.php
galette/lib/Galette/Entity/PdfModel.php
galette/lib/Galette/Entity/Reminder.php
galette/lib/Galette/Entity/SavedSearch.php
galette/lib/Galette/Entity/Status.php
galette/lib/Galette/Entity/Texts.php
galette/lib/Galette/Entity/Title.php
galette/lib/Galette/Entity/Transaction.php
galette/lib/Galette/Repository/Contributions.php
galette/lib/Galette/Repository/Groups.php
galette/lib/Galette/Repository/Members.php
galette/lib/Galette/Repository/PaymentTypes.php
galette/lib/Galette/Repository/SavedSearches.php
tests/Galette/Core/tests/units/History.php
tests/Galette/Entity/tests/units/Contribution.php
tests/Galette/Entity/tests/units/Transaction.php

index 219497a04066d33b3b46b1185467f3ab29f21e8a..b222f7ecc7d67549fa6549ec51f161e85c1d1307 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2009-2014 The Galette Team
+ * Copyright © 2009-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2009-2014 The Galette Team
+ * @copyright 2009-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2009-02-09
@@ -51,7 +51,7 @@ use Laminas\Db\Sql\Select;
  * @name      History
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2009-2014 The Galette Team
+ * @copyright 2009-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2009-02-09
@@ -157,7 +157,7 @@ class History
                 "An error occurred trying to add log entry. " . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
 
         return true;
@@ -192,7 +192,7 @@ class History
                 'Unable to flush logs. | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -226,7 +226,7 @@ class History
                 'Unable to get history. | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -277,6 +277,7 @@ class History
                 'Cannot list actions from history! | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -353,6 +354,7 @@ class History
                 __METHOD__ . ' | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -389,7 +391,7 @@ class History
                 'Cannot count history | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
index 04945ee634c357fdb7be8698e2847b406769b112..26a8faa7ce53c6dbe3d7e5401fe6bd6e4bd14f8f 100644 (file)
@@ -3,12 +3,12 @@
 /* vim: set expandtab tabstop=4 shiftwidth=4 softtabstop=4: */
 
 /**
- * Temporay links for galette, to send direct links to invoices, recipts,
+ * Temporary links for galette, to send direct links to invoices, receipts,
  * and member cards directly by email
  *
  * PHP version 5
  *
- * Copyright © 2020 The Galette Team
+ * Copyright © 2020-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -29,7 +29,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2020 The Galette Team
+ * @copyright 2020-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.9.4 - 2020-03-11
@@ -43,13 +43,14 @@ use Galette\Entity\Adherent;
 use Galette\Entity\Contribution;
 
 /**
- * Temporary password managment
+ * Temporary links for galette, to send direct links to invoices, receipts,
+ * and member cards directly by email
  *
  * @category  Core
  * @name      Links
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2020 The Galette Team
+ * @copyright 2020-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.9.4 - 2020-03-11
index 597bb207e255cc2b19b752634190f785c5b18320..028cfb30c0db9bff5128c5a7813df1c905bcc168 100644 (file)
@@ -208,7 +208,7 @@ class Login extends Authentication
     }
 
     /**
-     * Populate object after successfull login
+     * Populate object after successful login
      *
      * @param \ArrayObject $row User information
      *
index d2d3dbafcd1d0f9aee021a5b3512f197d3d8161a..34e2bd3fa7d1b869dc64dedb036034f8f942449b 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2009-2018 The Galette Team
+ * Copyright © 2009-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2011-2018 The Galette Team
+ * @copyright 2011-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2011-08-27
@@ -51,7 +51,7 @@ use Laminas\Db\Sql\Expression;
  * @name      MailingHistory
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2011-2018 The Galette Team
+ * @copyright 2011-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2011-08-27
@@ -167,7 +167,7 @@ class MailingHistory extends History
                 'Unable to get history. | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -203,6 +203,7 @@ class MailingHistory extends History
                 'Cannot list senders from mailing history! | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -299,6 +300,7 @@ class MailingHistory extends History
                 __METHOD__ . ' | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -335,7 +337,7 @@ class MailingHistory extends History
                 'Cannot count history | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -366,7 +368,7 @@ class MailingHistory extends History
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -408,7 +410,6 @@ class MailingHistory extends History
                 '] Mailing should be an instance of Mailing',
                 Analog::ERROR
             );
-            return false;
         }
     }
 
@@ -457,7 +458,7 @@ class MailingHistory extends History
                 'An error occurend updating Mailing | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -511,7 +512,7 @@ class MailingHistory extends History
                 'An error occurend storing Mailing | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 89dd0a57e1248653970b0a1e451736d18cdd9cbd..c6d9f6ae180c387310deef68be2da3f298a5f246 100644 (file)
@@ -314,7 +314,7 @@ class Adherent
                 'Cannot load member form id `' . $id . '` | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -348,7 +348,7 @@ class Adherent
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -497,7 +497,7 @@ class Adherent
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -858,7 +858,7 @@ class Adherent
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -932,7 +932,7 @@ class Adherent
                 '` | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -1602,7 +1602,7 @@ class Adherent
                 $e->getTraceAsString(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -1628,6 +1628,7 @@ class Adherent
                 $e->getMessage() . "\n" . $e->getTraceAsString(),
                 Analog::ERROR
             );
+            throw $e;
         }
     }
 
index 4dbd782cda29dbef7f776d19d4711e8015058e05..608d5de92215cdbc05d2676ee9363cfa9a829695 100644 (file)
@@ -300,6 +300,13 @@ class Contribution
             );
             //restrict query on current member id if he's not admin nor staff member
             if (!$this->login->isAdmin() && !$this->login->isStaff()) {
+                if (!$this->login->isLogged()) {
+                    Analog::log(
+                        'Non-logged-in users cannot load contribution id `' . $id,
+                        Analog::ERROR
+                    );
+                    return false;
+                }
                 if (!$this->login->isGroupManager()) {
                     $select->where
                         ->nest()
@@ -326,9 +333,11 @@ class Contribution
                 $this->loadFromRS($row);
                 return true;
             } else {
-                throw new \Exception(
-                    'No contribution #' . $id . ' (user ' . $this->login->id . ')'
+                Analog::log(
+                    'No contribution #' . $id . ' (user ' . $this->login->id . ')',
+                    Analog::ERROR
                 );
+                return false;
             }
         } catch (Throwable $e) {
             Analog::log(
@@ -336,7 +345,7 @@ class Contribution
                 $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -600,7 +609,7 @@ class Contribution
                 'An error occurred checking overlapping fee. ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -723,7 +732,7 @@ class Contribution
             if ($this->zdb->connection->inTransaction()) {
                 $this->zdb->connection->rollBack();
             }
-            return false;
+            throw $e;
         }
     }
 
@@ -758,7 +767,7 @@ class Contribution
                 $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -785,9 +794,11 @@ class Contribution
                 $this->updateDeadline();
                 $this->dynamicsRemove(true);
             } else {
-                throw new \RuntimeException(
-                    'Contribution has not been removed!'
+                Analog::log(
+                    'Contribution has not been removed!',
+                    Analog::WARNING
                 );
+                return false;
             }
             if ($transaction) {
                 $this->zdb->connection->commit();
@@ -803,7 +814,7 @@ class Contribution
                 $this->_id . ' | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -898,7 +909,7 @@ class Contribution
                 'An error occurred trying to retrieve member\'s due date',
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -940,7 +951,7 @@ class Contribution
                 ' to transaction #' . $trans_id . ' | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -969,7 +980,7 @@ class Contribution
                 ' to transaction #' . $trans_id . ' | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index c99da09157dee463a9f7880303119e82bffb7c95..85086139f9d92850f2082b37b8ce21e2be4c287c 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2007-2014 The Galette Team
+ * Copyright © 2007-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2007-2014 The Galette Team
+ * @copyright 2007-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2007-10-27
@@ -45,7 +45,7 @@ use Galette\Core\Db;
  * @name      ContibutionTypes
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2007-2014 The Galette Team
+ * @copyright 2007-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2007-10-27
@@ -81,7 +81,7 @@ class ContributionsTypes extends Entitled
      * Default constructor
      *
      * @param Db        $zdb  Database
-     * @param ResultSet $args Optionnal existing result set
+     * @param ResultSet $args Optional existing result set
      */
     public function __construct(Db $zdb, $args = null)
     {
index ea7212d907094de7a6d729bf9f8b1d24afa071c3..7b0376722ecc552052d98afc9e0240da0116a428 100644 (file)
@@ -166,7 +166,7 @@ class DynamicFieldsHandle
                 __METHOD__ . ' | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
index 2d58b0930b0144e488639783be35fe693b0ba12c..93295e1531dca95b868717b9055400e9a5b317ca 100644 (file)
@@ -135,9 +135,11 @@ abstract class Entitled
 
                 return true;
             } else {
-                throw new \RuntimeException(
-                    'Unknown ID ' . $id . '!'
+                Analog::log(
+                    'Unknown ID ' . $id,
+                    Analog::ERROR
                 );
+                return false;
             }
         } catch (Throwable $e) {
             Analog::log(
@@ -145,7 +147,7 @@ abstract class Entitled
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -169,7 +171,8 @@ abstract class Entitled
     /**
      * Set defaults at install time
      *
-     * @return boolean|Exception
+     * @return boolean
+     * @throws Throwable
      */
     public function installInit()
     {
@@ -277,7 +280,7 @@ abstract class Entitled
                 __METHOD__ . ' | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -322,7 +325,7 @@ abstract class Entitled
                 ') | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -358,7 +361,7 @@ abstract class Entitled
                 __METHOD__ . ' | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -410,7 +413,7 @@ abstract class Entitled
                 $label . '` | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 6e53097b97b91c787d3acaa86e27b75d8bbc3f9e..babf0b8ee552c8fbd1ceeb109bdb223df15b71d3 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2009-2014 The Galette Team
+ * Copyright © 2009-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2009-2014 The Galette Team
+ * @copyright 2009-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2009-03-28
@@ -47,7 +47,7 @@ use Galette\Core\Db;
  * @name      FieldsCategories
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2009-2014 The Galette Team
+ * @copyright 2009-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2009-03-28
@@ -103,7 +103,7 @@ class FieldsCategories
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -150,7 +150,8 @@ class FieldsCategories
      * Set default fields categories at install time
      *
      *
-     * @return boolean|Exception
+     * @return boolean
+     * @throws Throwable
      */
     public function installInit()
     {
index 656973f32be935b5193da7f9b17e6241c65a66d4..60434a581275753367cc7170533f27b9684de03f 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2009-2020 The Galette Team
+ * Copyright © 2009-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2009-2020 The Galette Team
+ * @copyright 2009-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2009-03-26
@@ -53,7 +53,7 @@ use Galette\Core\Authentication;
  * @name      FieldsConfig
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2009-2020 The Galette Team
+ * @copyright 2009-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2009-03-26
@@ -387,7 +387,8 @@ class FieldsConfig
      * Set default fields configuration at install time. All previous
      * existing values will be dropped first, including fields categories.
      *
-     * @return boolean|\Exception
+     * @return boolean
+     * @throws Throwable
      */
     public function installInit()
     {
@@ -804,10 +805,6 @@ class FieldsConfig
                 $e->getMessage(),
                 Analog::ERROR
             );
-            Analog::log(
-                $e->getTraceAsString(),
-                Analog::ERROR
-            );
             throw $e;
         }
     }
@@ -889,7 +886,7 @@ class FieldsConfig
                 $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 4c865622f9a6a1dd88e3936c35be5334df022e6f..088a5f29cb67c34165b8a2896f307745936b5ed4 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2012-2014 The Galette Team
+ * Copyright © 2012-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2012-2014 The Galette Team
+ * @copyright 2012-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2012-01-17
@@ -48,7 +48,7 @@ use Laminas\Db\Sql\Expression;
  * @name      Group
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2012-2014 The Galette Team
+ * @copyright 2012-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2012-01-17
@@ -120,7 +120,7 @@ class Group
                 'Cannot load group form id `' . $id . '` | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -209,6 +209,7 @@ class Group
                     'Cannot get group persons | ' . $e->getMessage(),
                     Analog::WARNING
                 );
+                throw $e;
             }
         }
     }
@@ -251,6 +252,7 @@ class Group
                 ' (' . $this->id . ')| ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -339,6 +341,7 @@ class Group
                     ' (' . $this->id . ') |' . $e->getMessage(),
                     Analog::ERROR
                 );
+                throw $e;
             }
             return false;
         }
@@ -392,7 +395,7 @@ class Group
                 $e->getTraceAsString(),
                 Analog::ERROR
             );
-            throw new \Exception(_T("Unable to detach group :("));
+            throw $e;
         }
     }
 
@@ -465,7 +468,7 @@ class Group
                 $e->getTraceAsString(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -795,7 +798,7 @@ class Group
                 '` (' . $this->id . ')|' . implode("\n", $messages),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -885,7 +888,7 @@ class Group
                 '` (' . $this->id . ')|' . implode("\n", $messages),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 3a3fd72c6497b18eedfbbc2e48030c88a3390a91..1c79023c1ada3dd70d5a610d437cee13ce78d4f9 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2013-2014 The Galette Team
+ * Copyright © 2013-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2013-2014 The Galette Team
+ * @copyright 2013-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7.6dev - 2013-09-26
@@ -47,7 +47,7 @@ use Laminas\Db\Adapter\Adapter;
  * @name      ImportModel
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2013-2014 The Galette Team
+ * @copyright 2013-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7.6dev - 2013-09-26
@@ -89,7 +89,7 @@ class ImportModel
                 "\n" . $e->__toString(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -135,7 +135,7 @@ class ImportModel
                 'Unable to remove import model ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -185,7 +185,7 @@ class ImportModel
                 $e->getMessage() . "\n" . $e->getTraceAsString(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index ff83eea7f84f50241abda54697afcf6fa6e6eaa0..256b4bafe6ae6edf1691965ded80ab27bffd1b74 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2020 The Galette Team
+ * Copyright © 2020-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2020 The Galette Team
+ * @copyright 2020-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.9.4dev - 2020-05-13
@@ -52,7 +52,7 @@ use Galette\Core\Authentication;
  * @name      FieldsConfig
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2020 The Galette Team
+ * @copyright 2020-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.9.4dev - 2020-05-13
@@ -355,11 +355,7 @@ class ListsConfig extends FieldsConfig
                 $e->getMessage(),
                 Analog::ERROR
             );
-            Analog::log(
-                $e->getTraceAsString(),
-                Analog::ERROR
-            );
-            return false;
+            throw $e;
         }
     }
 
index 1180f57197ef897f37de50fd2237ed272f198e6d..5d0d728643a783bde2c5554e32b918f4bf1ee325 100644 (file)
@@ -113,6 +113,7 @@ class PaymentType
                 $e->getMessage(),
                 Analog::ERROR
             );
+            throw $e;
         }
     }
 
@@ -172,7 +173,7 @@ class PaymentType
                 "\n" . print_r($data, true),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -298,7 +299,7 @@ class PaymentType
     }
 
     /**
-     * Is current paymant a system one
+     * Is current payment a system one
      *
      * @return boolean
      *
index a8b455a04232e0666e17e33b039760ad321b88ea..e4a9da8bd4e76b5daed5bcd4c9c9fbc0b47119cb 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2013-2014 The Galette Team
+ * Copyright © 2013-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Guillaume Rousse <guillomovitch@gmail.com>
- * @copyright 2013-2014 The Galette Team
+ * @copyright 2013-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.8.3dev - 2014-07-02
@@ -36,8 +36,6 @@
 
 namespace Galette\Entity;
 
-use Analog\Analog as Analog;
-
 /**
  * PDF form model
  *
@@ -45,7 +43,7 @@ use Analog\Analog as Analog;
  * @name      PdfAdhesionFormModel
  * @package   Galette
  * @author    Guillaume Rousse <guillomovitch@gmail.com>
- * @copyright 2013-2014 The Galette Team
+ * @copyright 2013-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.8.3dev - 2014-07-02
index 72e357b8fb029f349a7fbe2517b6f69a18a967ed..c833300cd6220b1e55bf702893dbeb84c0cd19b6 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2020 The Galette Team
+ * Copyright © 2020-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2020 The Galette Team
+ * @copyright 2020-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     2020-11-21
@@ -49,7 +49,7 @@ use NumberFormatter;
  * @name      PdfContribution
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2020 The Galette Team
+ * @copyright 2020-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     2020-11-21
index fb6d377759d6e183530024e037d54a26a6c034bb..49219641983b9116bb67630dd2333d3244381dd7 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2013-2020 The Galette Team
+ * Copyright © 2013-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2013-2020 The Galette Team
+ * @copyright 2013-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7.5dev - 2013-02-19
@@ -51,7 +51,7 @@ use Laminas\Db\Sql\Expression;
  * @name      PdfModel
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2013-2020 The Galette Team
+ * @copyright 2013-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7.5dev - 2013-02-19
@@ -251,7 +251,7 @@ abstract class PdfModel
                 "\n" . print_r($data, true),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 014ef5aa6f9f99f84a6dd0c04ce426c7da75026c..c949aef84a766c1a31f827a922a7fb556c16e74a 100644 (file)
@@ -121,6 +121,7 @@ class Reminder
                 $e->getMessage(),
                 Analog::ERROR
             );
+            throw $e;
         }
     }
 
@@ -149,10 +150,7 @@ class Reminder
                 __METHOD__ . ': incorrect ResultSet. Error: ' . $e->getMessage(),
                 Analog::ERROR
             );
-            Analog::log(
-                print_r($rs, true),
-                Analog::INFO
-            );
+            throw $e;
         }
     }
 
@@ -193,7 +191,7 @@ class Reminder
                 "\n" . print_r($data, true),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 56ba45b802e3b3870d1bb8cc25cb24198bbc6d59..6e97bead0283ab38c65a77733c13d05d34ed5cd2 100644 (file)
@@ -119,6 +119,7 @@ class SavedSearch
                 $e->getMessage(),
                 Analog::ERROR
             );
+            throw $e;
         }
     }
 
index 26230ec7a7e7d617a051e3899355e94ad2bd60ae..382087192c106bf82dd7df14a40a98635d52a006 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2007-2014 The Galette Team
+ * Copyright © 2007-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2007-2014 The Galette Team
+ * @copyright 2007-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2007-10-27
@@ -45,7 +45,7 @@ use Galette\Core\Db;
  * @name      Status
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2007-2014 The Galette Team
+ * @copyright 2007-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2007-10-27
@@ -83,7 +83,7 @@ class Status extends Entitled
      * Default constructor
      *
      * @param Db        $zdb  Database
-     * @param ResultSet $args Optionnal existing result set
+     * @param ResultSet $args Optional existing result set
      */
     public function __construct(Db $zdb, $args = null)
     {
index 8f59cfa21e430db99d438878fe93f7227b415f09..2b115fad90791366815f8c604bbf13f2c7d5d977 100644 (file)
@@ -327,7 +327,7 @@ class Texts
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -366,7 +366,7 @@ class Texts
                 $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -397,7 +397,7 @@ class Texts
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
index cfedaf5ff4eb795c5a36903ca14cf28edc42222f..1fb7a41adb159b42b40ac3f5508ca7b44266f796 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2013-2014 The Galette Team
+ * Copyright © 2013-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2013-2014 The Galette Team
+ * @copyright 2013-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7.4dev - 2013-01-27
@@ -46,7 +46,7 @@ use Analog\Analog;
  * @name      Title
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2009-2014 The Galette Team
+ * @copyright 2009-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.7dev - 2009-03-04
@@ -105,6 +105,7 @@ class Title
                 $e->getMessage(),
                 Analog::ERROR
             );
+            throw $e;
         }
     }
 
@@ -166,7 +167,7 @@ class Title
                 "\n" . print_r($data, true),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 1150040748920fcccdfaa2a6ff07418d89e33fcb..321db6e38504d75bb5a25ba0f76527f8662fe7bf 100644 (file)
@@ -163,6 +163,13 @@ class Transaction
 
             //restrict query on current member id if he's not admin nor staff member
             if (!$this->login->isAdmin() && !$this->login->isStaff() && !$this->login->isGroupManager()) {
+                if (!$this->login->isLogged()) {
+                    Analog::log(
+                        'Non-logged-in users cannot load transaction id `' . $id,
+                        Analog::ERROR
+                    );
+                    return false;
+                }
                 $select->where
                     ->nest()
                         ->equalTo('a.' . Adherent::PK, $this->login->id)
@@ -182,7 +189,11 @@ class Transaction
                 $this->loadFromRS($result);
                 return true;
             } else {
-                throw new \Exception();
+                Analog::log(
+                    'Transaction id `' . $id . '` does not exists',
+                    Analog::WARNING
+                );
+                return false;
             }
         } catch (Throwable $e) {
             Analog::log(
@@ -190,7 +201,7 @@ class Transaction
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -227,9 +238,16 @@ class Transaction
             $delete->where(
                 self::PK . ' = ' . $this->_id
             );
-            $this->zdb->execute($delete);
-
-            $this->dynamicsRemove(true);
+            $del = $this->zdb->execute($delete);
+            if ($del->count() > 0) {
+                $this->dynamicsRemove(true);
+            } else {
+                Analog::log(
+                    'Transaction has not been removed!',
+                    Analog::WARNING
+                );
+                return false;
+            }
 
             if ($transaction) {
                 $this->zdb->connection->commit();
@@ -246,7 +264,7 @@ class Transaction
                 $this->_id . ' | ' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -475,7 +493,7 @@ class Transaction
                 $e->getTraceAsString(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -508,6 +526,7 @@ class Transaction
                 $e->getMessage(),
                 Analog::ERROR
             );
+            throw $e;
         }
     }
 
@@ -540,6 +559,7 @@ class Transaction
                 $e->getMessage(),
                 Analog::ERROR
             );
+            throw $e;
         }
     }
 
index 4fa5292d64d004442ce92bc26bfe8c400395cddb..85144c9a602ce0f23941900347b5cec1acead04a 100644 (file)
@@ -179,7 +179,7 @@ class Contributions
                 'Cannot build SELECT clause for contributions | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -217,7 +217,7 @@ class Contributions
                 'Cannot count contributions | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -250,7 +250,7 @@ class Contributions
                 'Cannot calculate contributions sum | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -506,7 +506,7 @@ class Contributions
                     $e->getMessage(),
                     Analog::ERROR
                 );
-                return false;
+                throw $e;
             }
         } else {
             //not numeric and not an array: incorrect.
index 221eb32b4e1d8159d1b552711c6224bf819061a1..5fa8cf987a06a5ba8e45227768feda4f3abdfc4f 100644 (file)
@@ -108,6 +108,7 @@ class Groups
                 'Cannot list groups (simple) | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -180,6 +181,7 @@ class Groups
                 'Cannot list groups | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -239,7 +241,7 @@ class Groups
                 $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -356,7 +358,7 @@ class Groups
                 $msg . ' |' . implode("\n", $messages),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -423,6 +425,7 @@ class Groups
                 'Cannot list groups (simple) | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
index 51227cdc3507fad5a4edcf179a338730a1df5e2f..36d0deb92d05d1a934cf7574c5c30507ff1cde93 100644 (file)
@@ -432,6 +432,7 @@ class Members
                     $e->getMessage(),
                     Analog::ERROR
                 );
+                throw $e;
             }
             return false;
         }
@@ -500,7 +501,7 @@ class Members
                 . $with_photos . ') | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -575,6 +576,7 @@ class Members
                 'Cannot load members form ids array | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
@@ -879,7 +881,7 @@ class Members
                 'Cannot count members | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -1590,7 +1592,7 @@ class Members
                 'empty logins/passwords (' . $e->getMessage(),
                 Analog::ERROR
             );
-            return false;
+            throw $e;
         }
     }
 
index 3ad7a9aa4cdf3869a12a5a9da95c91872936e521..65312646a18fa3a25ca556e9383b6fd506b7cfeb 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2018 The Galette Team
+ * Copyright © 2018-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2018 The Galette Team
+ * @copyright 2018-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.9.2dev - 2018-07-23
@@ -48,7 +48,7 @@ use Galette\Entity\PaymentType;
  * @name      PaymentTypes
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2018 The Galette Team
+ * @copyright 2018-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     Available since 0.9.2dev - 2018-07-23
@@ -90,6 +90,7 @@ class PaymentTypes extends Repository
                 'Cannot list payment types | ' . $e->getMessage(),
                 Analog::WARNING
             );
+            throw $e;
         }
     }
 
index a607bcae5cdc357eb4fcec124adce0a8b7865335..81972f24e98734ba53f2a7f974464da12b3180f3 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2019 The Galette Team
+ * Copyright © 2019-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   Galette
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2019 The Galette Team
+ * @copyright 2019-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     2019-09-21
@@ -53,7 +53,7 @@ use Galette\Entity\Adherent;
  * @name      SavedSearches
  * @package   Galette
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2019 The Galette Team
+ * @copyright 2019-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @link      http://galette.tuxfamily.org
  * @since     2019-09-21
@@ -157,7 +157,7 @@ class SavedSearches
                 'Cannot build SELECT clause for saved searches | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -195,7 +195,7 @@ class SavedSearches
                 'Cannot count saved searches | ' . $e->getMessage(),
                 Analog::WARNING
             );
-            return false;
+            throw $e;
         }
     }
 
@@ -277,7 +277,7 @@ class SavedSearches
                     $e->getMessage(),
                     Analog::ERROR
                 );
-                return false;
+                throw $e;
             }
         } else {
             //not numeric and not an array: incorrect.
index 497aebb36eab7d211e3ac4c5c5b4b06ff53bfec0..2cfacd794febf24d613da060e92cc8ad9c5f756a 100644 (file)
@@ -7,7 +7,7 @@
  *
  * PHP version 5
  *
- * Copyright © 2016 The Galette Team
+ * Copyright © 2016-2021 The Galette Team
  *
  * This file is part of Galette (http://galette.tuxfamily.org).
  *
@@ -28,7 +28,7 @@
  * @package   GaletteTests
  *
  * @author    Johan Cwiklinski <johan@x-tnd.be>
- * @copyright 2016 The Galette Team
+ * @copyright 2016-2021 The Galette Team
  * @license   http://www.gnu.org/licenses/gpl-3.0.html GPL License 3.0 or (at your option) any later version
  * @version   SVN: $Id$
  * @link      http://galette.tuxfamily.org
@@ -125,38 +125,4 @@ class History extends GaletteTestCase
 
         $this->cleanHistory();
     }
-
-    /**
-     * Test add that throws an exception
-     *
-     * @return void
-     */
-    public function testAddWException()
-    {
-        $this->zdb = new \mock\Galette\Core\Db();
-        $this->calling($this->zdb)->execute = function ($o) {
-            throw new \LogicException('Error executing query!', 123);
-        };
-
-        $this->history = new \Galette\Core\History($this->zdb, $this->login, $this->preferences);
-        $add = $this->history->add('Test');
-        $this->boolean($add)->isFalse();
-    }
-
-    /**
-     * Test getHistory that throws an exception
-     *
-     * @return void
-     */
-    public function testGetHistoryWException()
-    {
-        $this->zdb = new \mock\Galette\Core\Db();
-        $this->calling($this->zdb)->execute = function ($o) {
-            throw new \LogicException('Error executing query!', 123);
-        };
-
-        $this->history = new \Galette\Core\History($this->zdb, $this->login, $this->preferences);
-        $list = $this->history->getHistory();
-        $this->boolean($list)->isFalse();
-    }
 }
index 36cc820df17c170c5a94392ba33394ae5dc011d5..6e03bd068ef3462bd6e6422fc148e7ce2ef17fc2 100644 (file)
@@ -468,25 +468,6 @@ class Contribution extends GaletteTestCase
             ->message->startWith('Existing errors prevents storing contribution');
     }
 
-    /**
-     * Test checkOverlap method that throws an exception
-     *
-     * @return void
-     */
-    public function testCheckOverlapWException()
-    {
-        $zdb = new \mock\Galette\Core\Db();
-        $this->calling($zdb)->execute = function ($o) {
-            if ($o instanceof \Zend\Db\Sql\Select) {
-                throw new \LogicException('Error executing query!', 123);
-            }
-        };
-
-        $contrib = new \Galette\Entity\Contribution($zdb, $this->login);
-        $this->boolean($contrib->checkOverlap())->isFalse();
-    }
-
-
     /**
      * Test fields labels
      *
@@ -545,8 +526,6 @@ class Contribution extends GaletteTestCase
         $this->createContribution();
 
         $this->boolean($this->contrib->remove())->isTrue();
-
-        $contrib = new \Galette\Entity\Contribution($this->zdb, $this->login);
         $this->boolean($this->contrib->remove())->isFalse();
     }
 
index e4cfa742562f7593ac933652b8a26decad1b0e50..7040657bf31a0465d1316991543c52615f19adf8 100644 (file)
@@ -310,9 +310,7 @@ class Transaction extends GaletteTestCase
         $this->boolean($this->transaction->load($tid))->isTrue();
         $this->boolean($this->transaction->remove($this->history))->isTrue();
         $this->boolean($this->transaction->load($tid))->isFalse();
-
-        $transaction = new \Galette\Entity\Transaction($this->zdb, $this->login);
-        $this->boolean($transaction->remove($this->history))->isFalse();
+        $this->boolean($this->transaction->remove($this->history))->isFalse();
     }
 
     /**