More NPEs catched (TODO: Should be rewritten)
authorRoland Häder <roland@mxchange.org>
Fri, 21 Aug 2009 22:04:19 +0000 (22:04 +0000)
committerRoland Häder <roland@mxchange.org>
Fri, 21 Aug 2009 22:04:19 +0000 (22:04 +0000)
application/ship-simu/main/user/class_ShipSimuBaseUser.php

index 426edcb..ffc4c6e 100644 (file)
@@ -106,8 +106,16 @@ class ShipSimuBaseUser extends BaseUser implements Registerable, Updateable {
                // By default they want to help.
                $ifGovHelps = true;
 
-               // First get a government instance from registry
-               $governmentInstance = Registry::getRegistry()->getInstance('government');
+               // Init instance
+               $governmentInstance = null;
+
+               try {
+                       // First get a government instance from registry
+                       $governmentInstance = Registry::getRegistry()->getInstance('government');
+               } catch (NullPointerException $e) {
+                       // Instance not found in registry
+                       // @TODO We should log this exception later
+               }
 
                // Is it there?
                if (is_null($governmentInstance)) {
@@ -137,8 +145,16 @@ class ShipSimuBaseUser extends BaseUser implements Registerable, Updateable {
                // By default they want to help.
                $ifGovHelps = true;
 
-               // First get a government instance from registry
-               $governmentInstance = Registry::getRegistry()->getInstance('government');
+               // Init instance
+               $governmentInstance = null;
+
+               try {
+                       // First get a government instance from registry
+                       $governmentInstance = Registry::getRegistry()->getInstance('government');
+               } catch (NullPointerException $e) {
+                       // Instance not found in registry
+                       // @TODO We should log this exception later
+               }
 
                // Is it there?
                if (is_null($governmentInstance)) {
@@ -169,8 +185,16 @@ class ShipSimuBaseUser extends BaseUser implements Registerable, Updateable {
                // Per default the money bank cannot pay
                $bankLends = false;
 
-               // Get a money bank instance from registry
-               $bankInstance = Registry::getRegistry()->getInstance('money_bank');
+               // Init instance
+               $bankInstance = null;
+
+               try {
+                       // Get a money bank instance from registry
+                       $bankInstance = Registry::getRegistry()->getInstance('money_bank');
+               } catch (NullPointerException $e) {
+                       // Instance not found in registry
+                       // @TODO We should log this exception later
+               }
 
                // Is it there?
                if (is_null($bankInstance)) {
@@ -205,8 +229,16 @@ class ShipSimuBaseUser extends BaseUser implements Registerable, Updateable {
                // For default he can still get money
                $hasMaxCredits = false;
 
-               // Get a money bank instance from registry
-               $bankInstance = Registry::getRegistry()->getInstance('money_bank');
+               // Init instance
+               $bankInstance = null;
+
+               try {
+                       // Get a money bank instance from registry
+                       $bankInstance = Registry::getRegistry()->getInstance('money_bank');
+               } catch (NullPointerException $e) {
+                       // Instance not found in registry
+                       // @TODO We should log this exception later
+               }
 
                // Is it there?
                if (is_null($bankInstance)) {