GemsTracker/gemstracker-library

Method getModelFields has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getModelFields($context, $new)
    {
        $compareOptions = [
            'NOW' => $this->_('Now (actual current time)'),
            'TS' => $this->_('At track start data'),

Avoid unused local variables such as '$maxAge'.

Open

        $maxAge  = $this->_data['gcon_condition_text4'];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid unused parameters such as '$fieldData'.

Open

    public function isTrackValid(\Gems_Tracker_RespondentTrack $respTrack, array $fieldData = null)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Avoid unused local variables such as '$minAge'.

Open

        $minAge  = $this->_data['gcon_condition_text2'];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}