qcubed/framework

View on GitHub
includes/framework/QDateTime.class.php

Summary

Maintainability
F
1 wk
Test Coverage

File QDateTime.class.php has 610 lines of code (exceeds 250 allowed). Consider refactoring.
Open

<?php

    // These Aid with the PHP 5.2 DateTime error handling
    class QDateTimeNullException extends QCallerException {}
    function QDateTimeErrorHandler() {}
Severity: Major
Found in includes/framework/QDateTime.class.php - About 1 day to fix

    Function __construct has a Cognitive Complexity of 43 (exceeds 5 allowed). Consider refactoring.
    Open

            public function __construct($mixValue = null, DateTimeZone $objTimeZone = null, $intType = QDateTime::UnknownType) {
                if ($mixValue instanceof QDateTime) {
                    // Cloning from another QDateTime object
                    if ($objTimeZone)
                        throw new QCallerException('QDateTime cloning cannot take in a DateTimeZone parameter');
    Severity: Minor
    Found in includes/framework/QDateTime.class.php - About 6 hrs to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Function qFormat has a Cognitive Complexity of 35 (exceeds 5 allowed). Consider refactoring.
    Open

            public function qFormat($strFormat = null) {
                if ($this->blnDateNull && $this->blnTimeNull) {
                    return '';
                }
    
    
    Severity: Minor
    Found in includes/framework/QDateTime.class.php - About 5 hrs to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    QDateTime has 38 functions (exceeds 20 allowed). Consider refactoring.
    Open

        class QDateTime extends DateTime implements JsonSerializable, Serializable {
            /** Used to specify the time right now (used when creating new instances of this class) */
            const Now = 'now';
            /** Date and time in ISO format */
            const FormatIso = 'YYYY-MM-DD hhhh:mm:ss';
    Severity: Minor
    Found in includes/framework/QDateTime.class.php - About 5 hrs to fix

      Method qFormat has 100 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

              public function qFormat($strFormat = null) {
                  if ($this->blnDateNull && $this->blnTimeNull) {
                      return '';
                  }
      
      
      Severity: Major
      Found in includes/framework/QDateTime.class.php - About 4 hrs to fix

        The class QDateTime has 27 non-getter- and setter-methods. Consider refactoring QDateTime to keep number of methods under 25.
        Open

            class QDateTime extends DateTime implements JsonSerializable, Serializable {
                /** Used to specify the time right now (used when creating new instances of this class) */
                const Now = 'now';
                /** Date and time in ISO format */
                const FormatIso = 'YYYY-MM-DD hhhh:mm:ss';

        TooManyMethods

        Since: 0.1

        A class with too many methods is probably a good suspect for refactoring, in order to reduce its complexity and find a way to have more fine grained objects.

        By default it ignores methods starting with 'get' or 'set'.

        The default was changed from 10 to 25 in PHPMD 2.3.

        Example

        Source https://phpmd.org/rules/codesize.html#toomanymethods

        The class QDateTime has 26 public methods. Consider refactoring QDateTime to keep number of public methods under 10.
        Open

            class QDateTime extends DateTime implements JsonSerializable, Serializable {
                /** Used to specify the time right now (used when creating new instances of this class) */
                const Now = 'now';
                /** Date and time in ISO format */
                const FormatIso = 'YYYY-MM-DD hhhh:mm:ss';

        TooManyPublicMethods

        Since: 0.1

        A class with too many public methods is probably a good suspect for refactoring, in order to reduce its complexity and find a way to have more fine grained objects.

        By default it ignores methods starting with 'get' or 'set'.

        Example

        Source https://phpmd.org/rules/codesize.html#toomanypublicmethods

        The class QDateTime has an overall complexity of 169 which is very high. The configured complexity threshold is 50.
        Open

            class QDateTime extends DateTime implements JsonSerializable, Serializable {
                /** Used to specify the time right now (used when creating new instances of this class) */
                const Now = 'now';
                /** Date and time in ISO format */
                const FormatIso = 'YYYY-MM-DD hhhh:mm:ss';

        Method __set has 81 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

                public function __set($strName, $mixValue) {
                    try {
                        switch ($strName) {
                            case 'Month':
                                if ($this->blnDateNull && (!is_null($mixValue)))
        Severity: Major
        Found in includes/framework/QDateTime.class.php - About 3 hrs to fix

          Function __get has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.
          Open

                  public function __get($strName) {
                      switch ($strName) {
                          case 'Month':
                              if ($this->blnDateNull)
                                  return null;
          Severity: Minor
          Found in includes/framework/QDateTime.class.php - About 2 hrs to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Method __construct has 55 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

                  public function __construct($mixValue = null, DateTimeZone $objTimeZone = null, $intType = QDateTime::UnknownType) {
                      if ($mixValue instanceof QDateTime) {
                          // Cloning from another QDateTime object
                          if ($objTimeZone)
                              throw new QCallerException('QDateTime cloning cannot take in a DateTimeZone parameter');
          Severity: Major
          Found in includes/framework/QDateTime.class.php - About 2 hrs to fix

            Method __get has 49 lines of code (exceeds 25 allowed). Consider refactoring.
            Open

                    public function __get($strName) {
                        switch ($strName) {
                            case 'Month':
                                if ($this->blnDateNull)
                                    return null;
            Severity: Minor
            Found in includes/framework/QDateTime.class.php - About 1 hr to fix

              Function __set has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.
              Open

                      public function __set($strName, $mixValue) {
                          try {
                              switch ($strName) {
                                  case 'Month':
                                      if ($this->blnDateNull && (!is_null($mixValue)))
              Severity: Minor
              Found in includes/framework/QDateTime.class.php - About 1 hr to fix

              Cognitive Complexity

              Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

              A method's cognitive complexity is based on a few simple rules:

              • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
              • Code is considered more complex for each "break in the linear flow of the code"
              • Code is considered more complex when "flow breaking structures are nested"

              Further reading

              Function setTime has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.
              Open

                      public function setTime($mixValue, $intMinute = null, $intSecond = null, $intMicroSeconds = null) {
                          if ($mixValue instanceof QDateTime) {
                              if ($mixValue->IsTimeNull()) {
                                  $this->blnTimeNull = true;
                                  $this->ReinforceNullProperties();
              Severity: Minor
              Found in includes/framework/QDateTime.class.php - About 1 hr to fix

              Cognitive Complexity

              Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

              A method's cognitive complexity is based on a few simple rules:

              • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
              • Code is considered more complex for each "break in the linear flow of the code"
              • Code is considered more complex when "flow breaking structures are nested"

              Further reading

              Method setTime has 36 lines of code (exceeds 25 allowed). Consider refactoring.
              Open

                      public function setTime($mixValue, $intMinute = null, $intSecond = null, $intMicroSeconds = null) {
                          if ($mixValue instanceof QDateTime) {
                              if ($mixValue->IsTimeNull()) {
                                  $this->blnTimeNull = true;
                                  $this->ReinforceNullProperties();
              Severity: Minor
              Found in includes/framework/QDateTime.class.php - About 1 hr to fix

                Avoid too many return statements within this method.
                Open

                                    return (int) parent::format('U'); // range depends on the platform's max and min integer values
                Severity: Major
                Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                  Avoid too many return statements within this method.
                  Open

                                      return self::FirstDayOfTheMonth($this->Month, $this->Year);
                  Severity: Major
                  Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                    Avoid too many return statements within this method.
                    Open

                                            return $mixValue;
                    Severity: Major
                    Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                      Avoid too many return statements within this method.
                      Open

                                              return $dtsFromCurrent->SimpleDisplay() . QApplication::Translate(' from now');
                      Severity: Major
                      Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                        Avoid too many return statements within this method.
                        Open

                                                return $mixValue;
                        Severity: Major
                        Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                          Avoid too many return statements within this method.
                          Open

                                                      return null;
                          Severity: Major
                          Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                            Avoid too many return statements within this method.
                            Open

                                                    return $mixValue;
                            Severity: Major
                            Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                              Avoid too many return statements within this method.
                              Open

                                                      return $mixValue;
                              Severity: Major
                              Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                Avoid too many return statements within this method.
                                Open

                                                        return (int) parent::format('H');
                                Severity: Major
                                Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                  Avoid too many return statements within this method.
                                  Open

                                                              return null;
                                  Severity: Major
                                  Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                    Avoid too many return statements within this method.
                                    Open

                                                            return null;
                                    Severity: Major
                                    Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                      Avoid too many return statements within this method.
                                      Open

                                                                  return null;
                                      Severity: Major
                                      Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                        Avoid too many return statements within this method.
                                        Open

                                                                return null;
                                        Severity: Major
                                        Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                          Avoid too many return statements within this method.
                                          Open

                                                                  return (int) parent::format('Y');
                                          Severity: Major
                                          Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                            Avoid too many return statements within this method.
                                            Open

                                                                    return null;
                                            Severity: Major
                                            Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                              Avoid too many return statements within this method.
                                              Open

                                                                      return (int) parent::format('i');
                                              Severity: Major
                                              Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                Avoid too many return statements within this method.
                                                Open

                                                                        return (int) parent::format('s');
                                                Severity: Major
                                                Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                  Avoid too many return statements within this method.
                                                  Open

                                                                          return $dtsFromCurrent->SimpleDisplay() . QApplication::Translate(' ago');
                                                  Severity: Major
                                                  Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                    Avoid too many return statements within this method.
                                                    Open

                                                                            return QApplication::Translate('right now');
                                                    Severity: Major
                                                    Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                      Avoid too many return statements within this method.
                                                      Open

                                                                          return self::LastDayOfTheMonth($this->Month, $this->Year);
                                                      Severity: Major
                                                      Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                        Avoid too many return statements within this method.
                                                        Open

                                                                                return null;
                                                        Severity: Major
                                                        Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                          Avoid too many return statements within this method.
                                                          Open

                                                                                  return $mixValue;
                                                          Severity: Major
                                                          Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                            Avoid too many return statements within this method.
                                                            Open

                                                                                        return null;
                                                            Severity: Major
                                                            Found in includes/framework/QDateTime.class.php - About 30 mins to fix

                                                              The method qFormat() has 129 lines of code. Current threshold is set to 100. Avoid really long methods.
                                                              Open

                                                                      public function qFormat($strFormat = null) {
                                                                          if ($this->blnDateNull && $this->blnTimeNull) {
                                                                              return '';
                                                                          }
                                                              
                                                              

                                                              The class QDateTime has 1080 lines of code. Current threshold is 1000. Avoid really long classes.
                                                              Open

                                                                  class QDateTime extends DateTime implements JsonSerializable, Serializable {
                                                                      /** Used to specify the time right now (used when creating new instances of this class) */
                                                                      const Now = 'now';
                                                                      /** Date and time in ISO format */
                                                                      const FormatIso = 'YYYY-MM-DD hhhh:mm:ss';

                                                              The method qFormat() has an NPath complexity of 936. The configured NPath complexity threshold is 200.
                                                              Open

                                                                      public function qFormat($strFormat = null) {
                                                                          if ($this->blnDateNull && $this->blnTimeNull) {
                                                                              return '';
                                                                          }
                                                              
                                                              

                                                              NPathComplexity

                                                              Since: 0.1

                                                              The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

                                                              Example

                                                              class Foo {
                                                                  function bar() {
                                                                      // lots of complicated code
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/codesize.html#npathcomplexity

                                                              The method __set() has a Cyclomatic Complexity of 27. The configured cyclomatic complexity threshold is 10.
                                                              Open

                                                                      public function __set($strName, $mixValue) {
                                                                          try {
                                                                              switch ($strName) {
                                                                                  case 'Month':
                                                                                      if ($this->blnDateNull && (!is_null($mixValue)))

                                                              CyclomaticComplexity

                                                              Since: 0.1

                                                              Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

                                                              Example

                                                              // Cyclomatic Complexity = 11
                                                              class Foo {
                                                              1   public function example() {
                                                              2       if ($a == $b) {
                                                              3           if ($a1 == $b1) {
                                                                              fiddle();
                                                              4           } elseif ($a2 == $b2) {
                                                                              fiddle();
                                                                          } else {
                                                                              fiddle();
                                                                          }
                                                              5       } elseif ($c == $d) {
                                                              6           while ($c == $d) {
                                                                              fiddle();
                                                                          }
                                                              7        } elseif ($e == $f) {
                                                              8           for ($n = 0; $n < $h; $n++) {
                                                                              fiddle();
                                                                          }
                                                                      } else {
                                                                          switch ($z) {
                                                              9               case 1:
                                                                                  fiddle();
                                                                                  break;
                                                              10              case 2:
                                                                                  fiddle();
                                                                                  break;
                                                              11              case 3:
                                                                                  fiddle();
                                                                                  break;
                                                                              default:
                                                                                  fiddle();
                                                                                  break;
                                                                          }
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

                                                              The method __construct() has a Cyclomatic Complexity of 13. The configured cyclomatic complexity threshold is 10.
                                                              Open

                                                                      public function __construct($mixValue = null, DateTimeZone $objTimeZone = null, $intType = QDateTime::UnknownType) {
                                                                          if ($mixValue instanceof QDateTime) {
                                                                              // Cloning from another QDateTime object
                                                                              if ($objTimeZone)
                                                                                  throw new QCallerException('QDateTime cloning cannot take in a DateTimeZone parameter');

                                                              CyclomaticComplexity

                                                              Since: 0.1

                                                              Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

                                                              Example

                                                              // Cyclomatic Complexity = 11
                                                              class Foo {
                                                              1   public function example() {
                                                              2       if ($a == $b) {
                                                              3           if ($a1 == $b1) {
                                                                              fiddle();
                                                              4           } elseif ($a2 == $b2) {
                                                                              fiddle();
                                                                          } else {
                                                                              fiddle();
                                                                          }
                                                              5       } elseif ($c == $d) {
                                                              6           while ($c == $d) {
                                                                              fiddle();
                                                                          }
                                                              7        } elseif ($e == $f) {
                                                              8           for ($n = 0; $n < $h; $n++) {
                                                                              fiddle();
                                                                          }
                                                                      } else {
                                                                          switch ($z) {
                                                              9               case 1:
                                                                                  fiddle();
                                                                                  break;
                                                              10              case 2:
                                                                                  fiddle();
                                                                                  break;
                                                              11              case 3:
                                                                                  fiddle();
                                                                                  break;
                                                                              default:
                                                                                  fiddle();
                                                                                  break;
                                                                          }
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

                                                              The method __get() has a Cyclomatic Complexity of 19. The configured cyclomatic complexity threshold is 10.
                                                              Open

                                                                      public function __get($strName) {
                                                                          switch ($strName) {
                                                                              case 'Month':
                                                                                  if ($this->blnDateNull)
                                                                                      return null;

                                                              CyclomaticComplexity

                                                              Since: 0.1

                                                              Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

                                                              Example

                                                              // Cyclomatic Complexity = 11
                                                              class Foo {
                                                              1   public function example() {
                                                              2       if ($a == $b) {
                                                              3           if ($a1 == $b1) {
                                                                              fiddle();
                                                              4           } elseif ($a2 == $b2) {
                                                                              fiddle();
                                                                          } else {
                                                                              fiddle();
                                                                          }
                                                              5       } elseif ($c == $d) {
                                                              6           while ($c == $d) {
                                                                              fiddle();
                                                                          }
                                                              7        } elseif ($e == $f) {
                                                              8           for ($n = 0; $n < $h; $n++) {
                                                                              fiddle();
                                                                          }
                                                                      } else {
                                                                          switch ($z) {
                                                              9               case 1:
                                                                                  fiddle();
                                                                                  break;
                                                              10              case 2:
                                                                                  fiddle();
                                                                                  break;
                                                              11              case 3:
                                                                                  fiddle();
                                                                                  break;
                                                                              default:
                                                                                  fiddle();
                                                                                  break;
                                                                          }
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

                                                              The method qFormat() has a Cyclomatic Complexity of 34. The configured cyclomatic complexity threshold is 10.
                                                              Open

                                                                      public function qFormat($strFormat = null) {
                                                                          if ($this->blnDateNull && $this->blnTimeNull) {
                                                                              return '';
                                                                          }
                                                              
                                                              

                                                              CyclomaticComplexity

                                                              Since: 0.1

                                                              Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

                                                              Example

                                                              // Cyclomatic Complexity = 11
                                                              class Foo {
                                                              1   public function example() {
                                                              2       if ($a == $b) {
                                                              3           if ($a1 == $b1) {
                                                                              fiddle();
                                                              4           } elseif ($a2 == $b2) {
                                                                              fiddle();
                                                                          } else {
                                                                              fiddle();
                                                                          }
                                                              5       } elseif ($c == $d) {
                                                              6           while ($c == $d) {
                                                                              fiddle();
                                                                          }
                                                              7        } elseif ($e == $f) {
                                                              8           for ($n = 0; $n < $h; $n++) {
                                                                              fiddle();
                                                                          }
                                                                      } else {
                                                                          switch ($z) {
                                                              9               case 1:
                                                                                  fiddle();
                                                                                  break;
                                                              10              case 2:
                                                                                  fiddle();
                                                                                  break;
                                                              11              case 3:
                                                                                  fiddle();
                                                                                  break;
                                                                              default:
                                                                                  fiddle();
                                                                                  break;
                                                                          }
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

                                                              Missing class import via use statement (line '96', column '23').
                                                              Open

                                                                          $dttToReturn = new QDateTime(QDateTime::Now);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '271', column '15').
                                                              Open

                                                                          return new QDateTime($temp);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '1004', column '16').
                                                              Open

                                                                                  throw new QUndefinedPropertyException('GET', 'QDateTime', $strName);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '704', column '20').
                                                              Open

                                                                              $dttThat = new QDateTime($dttCompare);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '651', column '20').
                                                              Open

                                                                              $dttThis = new QDateTime($this);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '1034', column '18').
                                                              Open

                                                                                          throw new QDateTimeNullException('Cannot set the Day property on a null date.  Use SetDate().');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '322', column '15').
                                                              Open

                                                                              $tz = new DateTimeZone($tz);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '1070', column '18').
                                                              Open

                                                                                          throw new QDateTimeNullException('Cannot set the Minute property on a null time.  Use SetTime().');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '110', column '18').
                                                              Open

                                                                          $dttNow = new QDateTime(QDateTime::Now);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '652', column '20').
                                                              Open

                                                                              $dttThat = new QDateTime($dttCompare);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '626', column '20').
                                                              Open

                                                                              $dttThat = new QDateTime($dttCompare);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '187', column '16').
                                                              Open

                                                                                  throw new QCallerException('QDateTime cloning cannot take in a DateTimeZone parameter');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '606', column '27').
                                                              Open

                                                                              $dtzNewTimezone = new DateTimeZone($strTimezoneIdentifier);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '1100', column '17').
                                                              Open

                                                                                      throw new QUndefinedPropertyException('SET', 'QDateTime', $strName);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '729', column '20').
                                                              Open

                                                                              $dttThis = new QDateTime($this);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              The method Now has a boolean flag argument $blnTimeValue, which is a certain sign of a Single Responsibility Principle violation.
                                                              Open

                                                                      public static function Now($blnTimeValue = true) {

                                                              BooleanArgumentFlag

                                                              Since: 1.4.0

                                                              A boolean flag argument is a reliable indicator for a violation of the Single Responsibility Principle (SRP). You can fix this problem by extracting the logic in the boolean flag into its own class or method.

                                                              Example

                                                              class Foo {
                                                                  public function bar($flag = true) {
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#booleanargumentflag

                                                              Missing class import via use statement (line '1022', column '18').
                                                              Open

                                                                                          throw new QDateTimeNullException('Cannot set the Month property on a null date.  Use SetDate().');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '1046', column '18').
                                                              Open

                                                                                          throw new QDateTimeNullException('Cannot set the Year property on a null date.  Use SetDate().');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '625', column '20').
                                                              Open

                                                                              $dttThis = new QDateTime($this);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '1058', column '18').
                                                              Open

                                                                                          throw new QDateTimeNullException('Cannot set the Hour property on a null time.  Use SetTime().');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '677', column '20').
                                                              Open

                                                                              $dttThis = new QDateTime($this);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '730', column '20').
                                                              Open

                                                                              $dttThat = new QDateTime($dttCompare);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '1082', column '18').
                                                              Open

                                                                                          throw new QDateTimeNullException('Cannot set the Second property on a null time.  Use SetTime().');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '168', column '15').
                                                              Open

                                                                          return new QDateTime(date('Y-m-d H:i:s', $intTimestamp), $objTimeZone);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '196', column '16').
                                                              Open

                                                                                  throw new QCallerException('QDateTime subclassing of a DateTime object cannot take in a DateTimeZone parameter');

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '678', column '20').
                                                              Open

                                                                              $dttThat = new QDateTime($dttCompare);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '760', column '20').
                                                              Open

                                                                              $dttThis = new QDateTime($this);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '703', column '20').
                                                              Open

                                                                              $dttThis = new QDateTime($this);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '796', column '15').
                                                              Open

                                                                              throw new QCallerException("Can only add DateTimeInterval or QDateTimeSpan objects");

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '283', column '15').
                                                              Open

                                                                          return new QDateTime($temp);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '780', column '15').
                                                              Open

                                                                          return new QDateTimeSpan($intDifference);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              Missing class import via use statement (line '761', column '20').
                                                              Open

                                                                              $dttThat = new QDateTime($dttCompare);

                                                              MissingImport

                                                              Since: 2.7.0

                                                              Importing all external classes in a file through use statements makes them clearly visible.

                                                              Example

                                                              function make() {
                                                                  return new \stdClass();
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#MissingImport

                                                              The method jsonSerialize uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              $dt = clone $this;
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method serialize uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              $strTz = null;
                                                                              $strDate = parent::format (DateTime::ISO8601);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method qFormat uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                              } else {
                                                                                  $strFormat = QDateTime::$DefaultFormat;
                                                                              }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method toJsObject uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              return sprintf ('new Date(%s, %s, %s, %s, %s, %s)', $dt->Year, $dt->Month - 1, $dt->Day, $dt->Hour, $dt->Minute, $dt->Second);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QDateTime' in method '__get'.
                                                              Open

                                                                                  $dtsFromCurrent = $this->Difference(QDateTime::Now());

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid using static access to class 'QType' in method 'setDate'.
                                                              Open

                                                                          $intMonth = QType::Cast($intMonth, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                              } else { // error
                                                                                  parent::__construct();
                                                                                  $this->blnDateNull = true;
                                                                                  $this->blnTimeNull = true;
                                                                              }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QType' in method 'setTime'.
                                                              Open

                                                                          $intHour = QType::Cast($intHour, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid using static access to class 'QType' in method 'setTime'.
                                                              Open

                                                                          $intMinute = QType::Cast($intMinute, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method setTime uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              parent::setTime($intHour, $intMinute, $intSecond);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method IsEarlierOrEqualTo uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              // Return the Result for the both Date and Time components
                                                                              return ($this->Timestamp <= $dttCompare->Timestamp);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method IsLaterThan uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              // Return the Result for the both Date and Time components
                                                                              return ($this->Timestamp > $dttCompare->Timestamp);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method jsonSerialize uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              return [JavaScriptHelper::ObjectType=>'qDateTime', 'year'=>$dt->Year, 'month'=>$dt->Month - 1,
                                                                                  'day'=>$dt->Day, 'hour'=>$dt->Hour, 'minute'=>$dt->Minute, 'second'=>$dt->Second];
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QApplication' in method '__get'.
                                                              Open

                                                                                      return $dtsFromCurrent->SimpleDisplay() . QApplication::Translate(' ago');

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid using static access to class 'QType' in method '__set'.
                                                              Open

                                                                                      $mixValue = QType::Cast($mixValue, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid using static access to class 'QType' in method '__set'.
                                                              Open

                                                                                      $mixValue = QType::Cast($mixValue, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                                  } else {
                                                                                      parent::__construct(date('Y-m-d H:i:s', $intTime), $objTimeZone);
                                                                                      $this->blnDateNull = false;
                                                                                      $this->blnTimeNull = false;
                                                                                  }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QApplication' in method '__get'.
                                                              Open

                                                                                      return QApplication::Translate('right now');

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          else {
                                                                              // string relative date or time
                                                                              if ($intTime = strtotime($mixValue)) {
                                                                                  // The documentation states that:
                                                                                  // The valid range of a timestamp is typically from 

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QType' in method 'setDate'.
                                                              Open

                                                                          $intYear = QType::Cast($intYear, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method IsEqualTo uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              // Return the Result for the both Date and Time components
                                                                              return ($this->Timestamp == $dttCompare->Timestamp);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QType' in method '__set'.
                                                              Open

                                                                                      $mixValue = QType::Cast($mixValue, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method setTime uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              $intHour = $mixValue;
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QType' in method '__set'.
                                                              Open

                                                                                      $mixValue = QType::Cast($mixValue, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid assigning values to variables in if clauses and the like (line '219', column '9').
                                                              Open

                                                                      public function __construct($mixValue = null, DateTimeZone $objTimeZone = null, $intType = QDateTime::UnknownType) {
                                                                          if ($mixValue instanceof QDateTime) {
                                                                              // Cloning from another QDateTime object
                                                                              if ($objTimeZone)
                                                                                  throw new QCallerException('QDateTime cloning cannot take in a DateTimeZone parameter');

                                                              IfStatementAssignment

                                                              Since: 2.7.0

                                                              Assignments in if clauses and the like are considered a code smell. Assignments in PHP return the right operand as their result. In many cases, this is an expected behavior, but can lead to many difficult to spot bugs, especially when the right operand could result in zero, null or an empty string and the like.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($foo = 'bar') { // possible typo
                                                                          // ...
                                                                      }
                                                                      if ($baz = 0) { // always false
                                                                          // ...
                                                                      }
                                                                  }
                                                              }

                                                              Source http://phpmd.org/rules/cleancode.html#ifstatementassignment

                                                              The method IsLaterOrEqualTo uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              // Return the Result for the both Date and Time components
                                                                              return ($this->Timestamp >= $dttCompare->Timestamp);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QApplication' in method '__get'.
                                                              Open

                                                                                      return $dtsFromCurrent->SimpleDisplay() . QApplication::Translate(' from now');

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method Compare uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              $dttThis = $this;
                                                                              $dttThat = $dttCompare;
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QType' in method '__set'.
                                                              Open

                                                                                      $mixValue = QType::Cast($mixValue, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid using static access to class 'QType' in method '__set'.
                                                              Open

                                                                                      $mixValue = QType::Cast($mixValue, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid using static access to class 'QType' in method 'setDate'.
                                                              Open

                                                                          $intDay = QType::Cast($intDay, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method toJsObject uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              $dt = clone $this;
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method unserialize uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              $tz = null;
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              The method setTime uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                              } else {
                                                                                  parent::setTime($intHour, $intMinute, $intSecond);
                                                                              }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QType' in method 'setTime'.
                                                              Open

                                                                          $intSecond = QType::Cast($intSecond, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              The method IsEarlierThan uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
                                                              Open

                                                                          } else {
                                                                              // Return the Result for the both Date and Time components
                                                                              return ($this->Timestamp < $dttCompare->Timestamp);
                                                                          }

                                                              ElseExpression

                                                              Since: 1.4.0

                                                              An if expression with an else branch is basically not necessary. You can rewrite the conditions in a way that the else clause is not necessary and the code becomes simpler to read. To achieve this, use early return statements, though you may need to split the code it several smaller methods. For very simple assignments you could also use the ternary operations.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar($flag)
                                                                  {
                                                                      if ($flag) {
                                                                          // one branch
                                                                      } else {
                                                                          // another branch
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#elseexpression

                                                              Avoid using static access to class 'QType' in method '__set'.
                                                              Open

                                                                                      $mixValue = QType::Cast($mixValue, QType::Integer);

                                                              StaticAccess

                                                              Since: 1.4.0

                                                              Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.

                                                              Example

                                                              class Foo
                                                              {
                                                                  public function bar()
                                                                  {
                                                                      Bar::baz();
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/cleancode.html#staticaccess

                                                              Avoid using empty try-catch blocks in ConvertToTimezone.
                                                              Open

                                                                          } catch (Exception $objExc) {}

                                                              EmptyCatchBlock

                                                              Since: 2.7.0

                                                              Usually empty try-catch is a bad idea because you are silently swallowing an error condition and then continuing execution. Occasionally this may be the right thing to do, but often it's a sign that a developer saw an exception, didn't know what to do about it, and so used an empty catch to silence the problem.

                                                              Example

                                                              class Foo {
                                                              
                                                                public function bar()
                                                                {
                                                                    try {
                                                                        // ...
                                                                    } catch (Exception $e) {} // empty catch block
                                                                }
                                                              }

                                                              Source https://phpmd.org/rules/design.html#emptycatchblock

                                                              Avoid using count() function in for loops.
                                                              Open

                                                                          for ($intIndex = 0; $intIndex < count($strArray); $intIndex++) {
                                                                              $strToken = trim($strArray[$intIndex]);
                                                                              if ($strToken) {
                                                                                  $intEndPosition = strpos($strFormat, $strArray[$intIndex], $intStartPosition);
                                                                                  $strToReturn .= substr($strFormat, $intStartPosition, $intEndPosition - $intStartPosition);

                                                              CountInLoopExpression

                                                              Since: 2.7.0

                                                              Using count/sizeof in loops expressions is considered bad practice and is a potential source of many bugs, especially when the loop manipulates an array, as count happens on each iteration.

                                                              Example

                                                              class Foo {
                                                              
                                                                public function bar()
                                                                {
                                                                  $array = array();
                                                              
                                                                  for ($i = 0; count($array); $i++) {
                                                                    // ...
                                                                  }
                                                                }
                                                              }

                                                              Source https://phpmd.org/rules/design.html#countinloopexpression

                                                              Similar blocks of code found in 2 locations. Consider refactoring.
                                                              Open

                                                                                  case 'Second':
                                                                                      if ($this->blnTimeNull && (!is_null($mixValue)))
                                                                                          throw new QDateTimeNullException('Cannot set the Second property on a null time.  Use SetTime().');
                                                                                      if (is_null($mixValue)) {
                                                                                          $this->blnTimeNull = true;
                                                              Severity: Major
                                                              Found in includes/framework/QDateTime.class.php and 1 other location - About 2 hrs to fix
                                                              includes/framework/QDateTime.class.php on lines 1032..1042

                                                              Duplicated Code

                                                              Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                                              Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                                              When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                                              Tuning

                                                              This issue has a mass of 123.

                                                              We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                                              The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                                              If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                                              See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                                              Refactorings

                                                              Further Reading

                                                              Similar blocks of code found in 2 locations. Consider refactoring.
                                                              Open

                                                                                  case 'Hour':
                                                                                      if ($this->blnTimeNull && (!is_null($mixValue)))
                                                                                          throw new QDateTimeNullException('Cannot set the Hour property on a null time.  Use SetTime().');
                                                                                      if (is_null($mixValue)) {
                                                                                          $this->blnTimeNull = true;
                                                              Severity: Major
                                                              Found in includes/framework/QDateTime.class.php and 1 other location - About 2 hrs to fix
                                                              includes/framework/QDateTime.class.php on lines 1044..1054

                                                              Duplicated Code

                                                              Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                                              Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                                              When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                                              Tuning

                                                              This issue has a mass of 123.

                                                              We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                                              The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                                              If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                                              See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                                              Refactorings

                                                              Further Reading

                                                              Similar blocks of code found in 2 locations. Consider refactoring.
                                                              Open

                                                                                  case 'Year':
                                                                                      if ($this->blnDateNull && (!is_null($mixValue)))
                                                                                          throw new QDateTimeNullException('Cannot set the Year property on a null date.  Use SetDate().');
                                                                                      if (is_null($mixValue)) {
                                                                                          $this->blnDateNull = true;
                                                              Severity: Major
                                                              Found in includes/framework/QDateTime.class.php and 1 other location - About 2 hrs to fix
                                                              includes/framework/QDateTime.class.php on lines 1056..1066

                                                              Duplicated Code

                                                              Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                                              Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                                              When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                                              Tuning

                                                              This issue has a mass of 123.

                                                              We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                                              The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                                              If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                                              See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                                              Refactorings

                                                              Further Reading

                                                              Similar blocks of code found in 2 locations. Consider refactoring.
                                                              Open

                                                                                  case 'Minute':
                                                                                      if ($this->blnTimeNull && (!is_null($mixValue)))
                                                                                          throw new QDateTimeNullException('Cannot set the Minute property on a null time.  Use SetTime().');
                                                                                      if (is_null($mixValue)) {
                                                                                          $this->blnTimeNull = true;
                                                              Severity: Major
                                                              Found in includes/framework/QDateTime.class.php and 1 other location - About 2 hrs to fix
                                                              includes/framework/QDateTime.class.php on lines 1020..1030

                                                              Duplicated Code

                                                              Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                                              Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                                              When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                                              Tuning

                                                              This issue has a mass of 123.

                                                              We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                                              The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                                              If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                                              See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                                              Refactorings

                                                              Further Reading

                                                              Similar blocks of code found in 2 locations. Consider refactoring.
                                                              Open

                                                                                  case 'Day':
                                                                                      if ($this->blnDateNull && (!is_null($mixValue)))
                                                                                          throw new QDateTimeNullException('Cannot set the Day property on a null date.  Use SetDate().');
                                                                                      if (is_null($mixValue)) {
                                                                                          $this->blnDateNull = true;
                                                              Severity: Major
                                                              Found in includes/framework/QDateTime.class.php and 1 other location - About 2 hrs to fix
                                                              includes/framework/QDateTime.class.php on lines 1080..1090

                                                              Duplicated Code

                                                              Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                                              Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                                              When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                                              Tuning

                                                              This issue has a mass of 123.

                                                              We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                                              The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                                              If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                                              See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                                              Refactorings

                                                              Further Reading

                                                              Similar blocks of code found in 2 locations. Consider refactoring.
                                                              Open

                                                                                  case 'Month':
                                                                                      if ($this->blnDateNull && (!is_null($mixValue)))
                                                                                          throw new QDateTimeNullException('Cannot set the Month property on a null date.  Use SetDate().');
                                                                                      if (is_null($mixValue)) {
                                                                                          $this->blnDateNull = true;
                                                              Severity: Major
                                                              Found in includes/framework/QDateTime.class.php and 1 other location - About 2 hrs to fix
                                                              includes/framework/QDateTime.class.php on lines 1068..1078

                                                              Duplicated Code

                                                              Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                                              Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                                              When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                                              Tuning

                                                              This issue has a mass of 123.

                                                              We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                                              The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                                              If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                                              See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                                              Refactorings

                                                              Further Reading

                                                              Avoid excessively long variable names like $strTimezoneIdentifier. Keep variable name length under 20.
                                                              Open

                                                                      public function ConvertToTimezone($strTimezoneIdentifier) {

                                                              LongVariable

                                                              Since: 0.2

                                                              Detects when a field, formal or local variable is declared with a long name.

                                                              Example

                                                              class Something {
                                                                  protected $reallyLongIntName = -3; // VIOLATION - Field
                                                                  public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
                                                                      $otherReallyLongName = -5; // VIOLATION - Local
                                                                      for ($interestingIntIndex = 0; // VIOLATION - For
                                                                           $interestingIntIndex < 10;
                                                                           $interestingIntIndex++ ) {
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#longvariable

                                                              Avoid excessively long variable names like $DefaultDateOnlyFormat. Keep variable name length under 20.
                                                              Open

                                                                      public static $DefaultDateOnlyFormat = QDateTime::FormatDisplayDate;

                                                              LongVariable

                                                              Since: 0.2

                                                              Detects when a field, formal or local variable is declared with a long name.

                                                              Example

                                                              class Something {
                                                                  protected $reallyLongIntName = -3; // VIOLATION - Field
                                                                  public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
                                                                      $otherReallyLongName = -5; // VIOLATION - Local
                                                                      for ($interestingIntIndex = 0; // VIOLATION - For
                                                                           $interestingIntIndex < 10;
                                                                           $interestingIntIndex++ ) {
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#longvariable

                                                              Avoid variables with short names like $dt. Configured minimum length is 3.
                                                              Open

                                                                              $dt = self::Now();    // time only will use today's date.

                                                              ShortVariable

                                                              Since: 0.2

                                                              Detects when a field, local, or parameter has a very short name.

                                                              Example

                                                              class Something {
                                                                  private $q = 15; // VIOLATION - Field
                                                                  public static function main( array $as ) { // VIOLATION - Formal
                                                                      $r = 20 + $this->q; // VIOLATION - Local
                                                                      for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
                                                                          $r += $this->q;
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#shortvariable

                                                              Avoid variables with short names like $s. Configured minimum length is 3.
                                                              Open

                                                                      public function unserialize($s) {

                                                              ShortVariable

                                                              Since: 0.2

                                                              Detects when a field, local, or parameter has a very short name.

                                                              Example

                                                              class Something {
                                                                  private $q = 15; // VIOLATION - Field
                                                                  public static function main( array $as ) { // VIOLATION - Formal
                                                                      $r = 20 + $this->q; // VIOLATION - Local
                                                                      for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
                                                                          $r += $this->q;
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#shortvariable

                                                              Avoid variables with short names like $tz. Configured minimum length is 3.
                                                              Open

                                                                              $tz = $this->getTimezone();

                                                              ShortVariable

                                                              Since: 0.2

                                                              Detects when a field, local, or parameter has a very short name.

                                                              Example

                                                              class Something {
                                                                  private $q = 15; // VIOLATION - Field
                                                                  public static function main( array $as ) { // VIOLATION - Formal
                                                                      $r = 20 + $this->q; // VIOLATION - Local
                                                                      for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
                                                                          $r += $this->q;
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#shortvariable

                                                              Avoid variables with short names like $dt. Configured minimum length is 3.
                                                              Open

                                                                              $dt = self::Now();    // time only will use today's date.

                                                              ShortVariable

                                                              Since: 0.2

                                                              Detects when a field, local, or parameter has a very short name.

                                                              Example

                                                              class Something {
                                                                  private $q = 15; // VIOLATION - Field
                                                                  public static function main( array $as ) { // VIOLATION - Formal
                                                                      $r = 20 + $this->q; // VIOLATION - Local
                                                                      for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
                                                                          $r += $this->q;
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#shortvariable

                                                              Avoid variables with short names like $tz. Configured minimum length is 3.
                                                              Open

                                                                          $tz = $this->getTimezone();

                                                              ShortVariable

                                                              Since: 0.2

                                                              Detects when a field, local, or parameter has a very short name.

                                                              Example

                                                              class Something {
                                                                  private $q = 15; // VIOLATION - Field
                                                                  public static function main( array $as ) { // VIOLATION - Formal
                                                                      $r = 20 + $this->q; // VIOLATION - Local
                                                                      for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
                                                                          $r += $this->q;
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#shortvariable

                                                              Avoid variables with short names like $tz. Configured minimum length is 3.
                                                              Open

                                                                          $tz = $a[4];

                                                              ShortVariable

                                                              Since: 0.2

                                                              Detects when a field, local, or parameter has a very short name.

                                                              Example

                                                              class Something {
                                                                  private $q = 15; // VIOLATION - Field
                                                                  public static function main( array $as ) { // VIOLATION - Formal
                                                                      $r = 20 + $this->q; // VIOLATION - Local
                                                                      for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
                                                                          $r += $this->q;
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#shortvariable

                                                              Avoid variables with short names like $a. Configured minimum length is 3.
                                                              Open

                                                                          $a = unserialize($s);

                                                              ShortVariable

                                                              Since: 0.2

                                                              Detects when a field, local, or parameter has a very short name.

                                                              Example

                                                              class Something {
                                                                  private $q = 15; // VIOLATION - Field
                                                                  public static function main( array $as ) { // VIOLATION - Formal
                                                                      $r = 20 + $this->q; // VIOLATION - Local
                                                                      for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
                                                                          $r += $this->q;
                                                                      }
                                                                  }
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#shortvariable

                                                              Constant UnknownType should be defined in uppercase
                                                              Open

                                                                      const UnknownType = 0;

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatDisplayTime should be defined in uppercase
                                                              Open

                                                                      const FormatDisplayTime = 'hh:mm:ss zz';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatSoap should be defined in uppercase
                                                              Open

                                                                      const FormatSoap = 'YYYY-MM-DDThhhh:mm:ss';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatIsoCompressed should be defined in uppercase
                                                              Open

                                                                      const FormatIsoCompressed = 'YYYYMMDDhhhhmmss';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatDisplayDateFull should be defined in uppercase
                                                              Open

                                                                      const FormatDisplayDateFull = 'DDD, MMMM D, YYYY';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant DateAndTimeType should be defined in uppercase
                                                              Open

                                                                      const DateAndTimeType = 3;

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatRfc5322 should be defined in uppercase
                                                              Open

                                                                      const FormatRfc5322 = 'DDD, DD MMM YYYY hhhh:mm:ss ttttt';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatDisplayDate should be defined in uppercase
                                                              Open

                                                                      const FormatDisplayDate = 'MMM DD YYYY';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatDisplayDateTimeFull should be defined in uppercase
                                                              Open

                                                                      const FormatDisplayDateTimeFull = 'DDDD, MMMM D, YYYY, h:mm:ss zz';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatDisplayDateTime should be defined in uppercase
                                                              Open

                                                                      const FormatDisplayDateTime = 'MMM DD YYYY hh:mm zz';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatRfc822 should be defined in uppercase
                                                              Open

                                                                      const FormatRfc822 = 'DDD, DD MMM YYYY hhhh:mm:ss ttt';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant TimeOnlyType should be defined in uppercase
                                                              Open

                                                                      const TimeOnlyType = 2;

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant Now should be defined in uppercase
                                                              Open

                                                                      const Now = 'now';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant FormatIso should be defined in uppercase
                                                              Open

                                                                      const FormatIso = 'YYYY-MM-DD hhhh:mm:ss';

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              Constant DateOnlyType should be defined in uppercase
                                                              Open

                                                                      const DateOnlyType = 1;

                                                              ConstantNamingConventions

                                                              Since: 0.2

                                                              Class/Interface constant names should always be defined in uppercase.

                                                              Example

                                                              class Foo {
                                                                  const MY_NUM = 0; // ok
                                                                  const myTest = ""; // fail
                                                              }

                                                              Source https://phpmd.org/rules/naming.html#constantnamingconventions

                                                              The method Modify is not named in camelCase.
                                                              Open

                                                                      public function Modify($mixValue) {
                                                                          parent::modify($mixValue);
                                                                          return $this;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method GetSoapDateTimeArray is not named in camelCase.
                                                              Open

                                                                      public function GetSoapDateTimeArray($dttArray) {
                                                                          if (!$dttArray)
                                                                              return null;
                                                              
                                                                          $strArrayToReturn = array();

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method Add is not named in camelCase.
                                                              Open

                                                                      public function Add($dtsSpan){
                                                                          if ($dtsSpan instanceof DateInterval) {
                                                                              parent::add($dtsSpan);
                                                                              return $this;
                                                                          }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method FromTimestamp is not named in camelCase.
                                                              Open

                                                                      public static function FromTimestamp($intTimestamp, DateTimeZone $objTimeZone = null) {
                                                                          return new QDateTime(date('Y-m-d H:i:s', $intTimestamp), $objTimeZone);
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsDateNull is not named in camelCase.
                                                              Open

                                                                      public function IsDateNull() {
                                                                          return $this->blnDateNull;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method AddHours is not named in camelCase.
                                                              Open

                                                                      public function AddHours($intHours){
                                                                          $this->Hour += $intHours;
                                                                          return $this;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method AddMonths is not named in camelCase.
                                                              Open

                                                                      public function AddMonths($intMonths){
                                                                          $prevDay = $this->Day;
                                                                          $this->Month += $intMonths;
                                                                          if ($this->Day != $prevDay) {
                                                                              $this->Day = 1;

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsLaterThan is not named in camelCase.
                                                              Open

                                                                      public function IsLaterThan(QDateTime $dttCompare) {
                                                                          // All comparison operations MUST have operands with matching Date Nullstates
                                                                          if ($this->blnDateNull != $dttCompare->blnDateNull)
                                                                              return false;
                                                              
                                                              

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method AddMinutes is not named in camelCase.
                                                              Open

                                                                      public function AddMinutes($intMinutes){
                                                                          $this->Minute += $intMinutes;
                                                                          return $this;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method PhpDate is not named in camelCase.
                                                              Open

                                                                      public function PhpDate($strFormat) {
                                                                          // This just makes a call to format
                                                                          return parent::format($strFormat);
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsLaterOrEqualTo is not named in camelCase.
                                                              Open

                                                                      public function IsLaterOrEqualTo(QDateTime $dttCompare) {
                                                                          // All comparison operations MUST have operands with matching Date Nullstates
                                                                          if ($this->blnDateNull != $dttCompare->blnDateNull)
                                                                              return false;
                                                              
                                                              

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method Difference is not named in camelCase.
                                                              Open

                                                                      public function Difference(QDateTime $dttDateTime) {
                                                                          $intDifference = $this->Timestamp - $dttDateTime->Timestamp;
                                                                          return new QDateTimeSpan($intDifference);
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method AddDays is not named in camelCase.
                                                              Open

                                                                      public function AddDays($intDays){
                                                                          $this->Day += $intDays;
                                                                          return $this;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsEarlierThan is not named in camelCase.
                                                              Open

                                                                      public function IsEarlierThan(QDateTime $dttCompare) {
                                                                          // All comparison operations MUST have operands with matching Date Nullstates
                                                                          if ($this->blnDateNull != $dttCompare->blnDateNull)
                                                                              return false;
                                                              
                                                              

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method AddSeconds is not named in camelCase.
                                                              Open

                                                                      public function AddSeconds($intSeconds){
                                                                          $this->Second += $intSeconds;
                                                                          return $this;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method Now is not named in camelCase.
                                                              Open

                                                                      public static function Now($blnTimeValue = true) {
                                                                          $dttToReturn = new QDateTime(QDateTime::Now);
                                                                          if (!$blnTimeValue) {
                                                                              $dttToReturn->blnTimeNull = true;
                                                                              $dttToReturn->ReinforceNullProperties();

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsEarlierOrEqualTo is not named in camelCase.
                                                              Open

                                                                      public function IsEarlierOrEqualTo(QDateTime $dttCompare) {
                                                                          // All comparison operations MUST have operands with matching Date Nullstates
                                                                          if ($this->blnDateNull != $dttCompare->blnDateNull)
                                                                              return false;
                                                              
                                                              

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method FirstDayOfTheMonth is not named in camelCase.
                                                              Open

                                                                      public static function FirstDayOfTheMonth($intMonth, $intYear) {
                                                                          $temp = date('Y-m-d',mktime(0,0,0,$intMonth,1,$intYear));
                                                                          return new QDateTime($temp);
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method LastDayOfTheMonth is not named in camelCase.
                                                              Open

                                                                      public static function LastDayOfTheMonth($intMonth, $intYear) {
                                                                          $temp = date('Y-m-t',mktime(0,0,0,$intMonth,1,$intYear));
                                                                          return new QDateTime($temp);
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsEqualTo is not named in camelCase.
                                                              Open

                                                                      public function IsEqualTo(QDateTime $dttCompare) {
                                                                          // All comparison operations MUST have operands with matching Date Nullstates
                                                                          if ($this->blnDateNull != $dttCompare->blnDateNull)
                                                                              return false;
                                                              
                                                              

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsNull is not named in camelCase.
                                                              Open

                                                                      public function IsNull() {
                                                                          return ($this->blnDateNull && $this->blnTimeNull);
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method ReinforceNullProperties is not named in camelCase.
                                                              Open

                                                                      protected function ReinforceNullProperties() {
                                                                          if ($this->blnDateNull)
                                                                              parent::setDate(2000, 1, 1);
                                                                          if ($this->blnTimeNull) {
                                                                              parent::setTime(0, 0, 0);

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method ConvertToTimezone is not named in camelCase.
                                                              Open

                                                                      public function ConvertToTimezone($strTimezoneIdentifier) {
                                                                          try {
                                                                              $dtzNewTimezone = new DateTimeZone($strTimezoneIdentifier);
                                                                              $this->SetTimezone($dtzNewTimezone);
                                                                          } catch (Exception $objExc) {}

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method Compare is not named in camelCase.
                                                              Open

                                                                      public function Compare(QDateTime $dttCompare) {
                                                                          // All comparison operations MUST have operands with matching Date Nullstates
                                                                          if ($this->blnDateNull && !$dttCompare->blnDateNull) {
                                                                              return -1;
                                                                          } elseif (!$this->blnDateNull && $dttCompare->blnDateNull) {

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method NowToString is not named in camelCase.
                                                              Open

                                                                      public static function NowToString($strFormat = null) {
                                                                          $dttNow = new QDateTime(QDateTime::Now);
                                                                          return $dttNow->qFormat($strFormat);
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method IsTimeNull is not named in camelCase.
                                                              Open

                                                                      public function IsTimeNull() {
                                                                          return $this->blnTimeNull;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              The method AddYears is not named in camelCase.
                                                              Open

                                                                      public function AddYears($intYears){
                                                                          $this->Year += $intYears;
                                                                          return $this;
                                                                      }

                                                              CamelCaseMethodName

                                                              Since: 0.2

                                                              It is considered best practice to use the camelCase notation to name methods.

                                                              Example

                                                              class ClassName {
                                                                  public function get_name() {
                                                                  }
                                                              }

                                                              Source

                                                              There are no issues that match your filters.

                                                              Category
                                                              Status