Method recorded_hours has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

Function recorded_hours has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

• Read up
  Read up

Avoid using undefined variables such as '$toProcessLen' which will lead to PHP notices.

Open

    $toProcessLen[] = $length;

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessUID' which will lead to PHP notices.

Open

    $toProcessUID[] = $user_id;

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessOID' which will lead to PHP notices.

Open

  $size = count($toProcessOID);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessOID' which will lead to PHP notices.

Open

  $size = count($toProcessOID);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessDID' which will lead to PHP notices.

Open

        WHERE d.detail_id = ".$toProcessDID[$i];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessOID' which will lead to PHP notices.

Open

  $size = count($toProcessOID);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessUID' which will lead to PHP notices.

Open

    $sql = "UPDATE service_attendance SET processed = 2 WHERE user_id = ".$toProcessUID[$i]." AND occurrence_id = ".$toProcessOID[$i];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$month' which will lead to PHP notices.

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$theDate' which will lead to PHP notices.

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$name' which will lead to PHP notices.

Open

      $name[] = $r['name'];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$day' which will lead to PHP notices.

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$db' which will lead to PHP notices.

Open

    $result = $db->query($sql);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$db' which will lead to PHP notices.

Open

    $result = $db->query($sql1);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$theDate' which will lead to PHP notices.

Open

      $theDate[] = $r['theDate'];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$servicetype' which will lead to PHP notices.

Open

        '".$servicetype[$i]."','".$name[$i]."')";

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessOID' which will lead to PHP notices.

Open

    $toProcessOID[] = $occurrence_id;

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessOID' which will lead to PHP notices.

Open

    $sql = "SELECT theDate FROM service_occurrence WHERE occurrence_id = ".$toProcessOID[$i];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$year' which will lead to PHP notices.

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessOID' which will lead to PHP notices.

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessUID' which will lead to PHP notices.

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$db' which will lead to PHP notices.

Open

    $result = $db->query($sql);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessLen' which will lead to PHP notices.

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$day' which will lead to PHP notices.

Open

      $day[] = substr($r['theDate'],-2);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$servicetype' which will lead to PHP notices.

Open

      $servicetype[] = $r['servicetype'];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$name' which will lead to PHP notices.

Open

        '".$servicetype[$i]."','".$name[$i]."')";

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$db' which will lead to PHP notices.

Open

  $result = $db->query($sql);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessOID' which will lead to PHP notices.

Open

    $sql = "UPDATE service_attendance SET processed = 2 WHERE user_id = ".$toProcessUID[$i]." AND occurrence_id = ".$toProcessOID[$i];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$db' which will lead to PHP notices.

Open

    $result = $db->query($sql);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$month' which will lead to PHP notices.

Open

      $month[] = substr($r['theDate'],-5,2);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$toProcessDID' which will lead to PHP notices.

Open

    $toProcessDID[] = $detail_id;

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$year' which will lead to PHP notices.

Open

      $year[] = substr($r['theDate'],0,4);

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 5 and the first side effect is on line 2.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

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

Open

  while($r = mysqli_fetch_array($result)){

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

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;
        }
    }
}

Line exceeds 120 characters; contains 134 characters

Open

    $sql = "UPDATE service_attendance SET processed = 2 WHERE user_id = ".$toProcessUID[$i]." AND occurrence_id = ".$toProcessOID[$i];

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 123 characters

Open

    $sql1 = "INSERT INTO recorded_hours (user_id, month, day, year, date, semester, description, hours, servicetype, event)

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 187 characters

Open

        VALUES (".$toProcessUID[$i].",".$month[$i].",".$day[$i].",".$year[$i].",'".$theDate[$i]."','".$current_semester."','auto-logged item #".$toProcessOID[$i]."',".$toProcessLen[$i].",

• Exclude checks
  • Disable engine
  • Disable check

Space before opening parenthesis of function call prohibited

Open

require_once ('session.php');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $length = $r['length'];

• Exclude checks
  • Disable engine
  • Disable check

Opening brace should be on a new line

Open

function recorded_hours(){

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after WHILE keyword; 0 found

Open

  while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $occurrence_id = $r['occurrence_id'];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  include('mysql_access.php');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  $sql = "SELECT user_id, detail_id, occurrence_id, length FROM service_attendance WHERE processed = 1";

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  global $current_semester;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $toProcessOID[] = $occurrence_id;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $toProcessLen[] = $length;

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing parenthesis; found 0

Open

  while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  $size = count($toProcessOID);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $toProcessDID[] = $detail_id;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $detail_id = $r['detail_id'];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  $result = $db->query($sql);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $toProcessUID[] = $user_id;

• Exclude checks
  • Disable engine
  • Disable check

Space before opening parenthesis of function call prohibited

Open

require_once ('mysql_access.php');

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after FOR keyword; 0 found

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $user_id = $r['user_id'];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $sql = "SELECT theDate FROM service_occurrence WHERE occurrence_id = ".$toProcessOID[$i];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing parenthesis; found 0

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      $theDate[] = $r['theDate'];

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after FOR keyword; 0 found

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      $day[] = substr($r['theDate'],-2);

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

      $day[] = substr($r['theDate'],-2);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

      $month[] = substr($r['theDate'],-5,2);

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

      $year[] = substr($r['theDate'],0,4);

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing parenthesis; found 0

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $sql = "SELECT d.length, e.name, e.servicetype

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    if(!$result){

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after WHILE keyword; 0 found

Open

    while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing parenthesis; found 0

Open

    while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      echo("something went wrong, contact the webmaster");

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

      $year[] = substr($r['theDate'],0,4);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      $name[] = $r['name'];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $result = $db->query($sql);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after IF keyword; 0 found

Open

    if(!$result){

• Exclude checks
  • Disable engine
  • Disable check

A closing tag is not permitted at the end of a PHP file

Open

?>

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      $month[] = substr($r['theDate'],-5,2);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      $year[] = substr($r['theDate'],0,4);

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing parenthesis; found 0

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      $servicetype[] = $r['servicetype'];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $sql = "UPDATE service_attendance SET processed = 2 WHERE user_id = ".$toProcessUID[$i]." AND occurrence_id = ".$toProcessOID[$i];

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing parenthesis; found 0

Open

    if(!$result){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  $size = count($toProcessOID);

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after closing parenthesis; found 0

Open

    while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

      $month[] = substr($r['theDate'],-5,2);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  $size = count($toProcessOID);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $sql1 = "INSERT INTO recorded_hours (user_id, month, day, year, date, semester, description, hours, servicetype, event)

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $result = $db->query($sql1);

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after FOR keyword; 0 found

Open

  for($i = 0; $i < $size; $i++){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $result = $db->query($sql);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $result = $db->query($sql);

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after WHILE keyword; 0 found

Open

    while($r = mysqli_fetch_array($result)){

• Exclude checks
  • Disable engine
  • Disable check

The variable $user_id is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $occurrence_id is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $user_id is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $occurrence_id is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $current_semester is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $detail_id is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $current_semester is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $detail_id is not named in camelCase.

Open

function recorded_hours(){
  include('mysql_access.php');
  global $current_semester;
  //this is shit code. could be written in OOP.
  // not shit code INVENTIVE code.

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

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}