Much simplier and with is_null() sanity check
authorRoland Häder <roland@mxchange.org>
Fri, 21 Aug 2009 21:55:43 +0000 (21:55 +0000)
committerRoland Häder <roland@mxchange.org>
Fri, 21 Aug 2009 21:55:43 +0000 (21:55 +0000)
inc/classes/main/helper/class_BaseHelper.php

index da4f6b1..5cd4ea8 100644 (file)
@@ -438,15 +438,17 @@ class BaseHelper extends BaseFrameworkSystem {
         *
         * @param       $fieldName              Name of the field we shall fetch
         * @return      $fieldValue             Value from field
+        * @throws      NullPointerException    Thrown if $valueInstance is null
         */
        public function getValueField ($fieldName) {
-               // Construct the eval() command
-               $eval = sprintf("\$fieldValue = \$this->getValueInstance()->getField('%s');",
-                       $fieldName
-               );
+               // The $valueInstance attribute should not be null!
+               if (is_null($this->getValueInstance())) {
+                       // Throws an exception here
+                       throw new NullPointerException($this, self::EXCEPTION_IS_NULL_POINTER);
+               } // END - if
 
                // Get the field value
-               eval($eval);
+               $fieldValue = $this->getValueInstance()->getField($fieldName);
 
                // Return it
                return $fieldValue;