Temporary_account_model

Complexity

Total Complexity

20

Size/Duplication

Total Lines	194
Duplicated Lines	29.9 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	hp ➔ randomPassword()	0	10	2
A	__construct()	0	4	1
B	create()	8	76	1
A	get_next_temp()	0	6	2
A	get_faculty()	13	13	3
A	get_department()	14	14	4
A	get_pending()	0	5	1
A	get_all()	0	4	1
A	check_status()	0	5	1
A	match_id_user()	0	5	1
A	cancel()	0	9	1
A	reject()	11	11	1
A	approve()	12	12	1
A	created_account()	0	9	1
A	get_account()	0	5	1

<?php defined('BASEPATH') OR exit('No direct script access allowed');

class Temporary_account_model extends CI_Model {

    public function __construct() {
        parent::__construct();
        $this->load->helper('date');
    }

    public function create($logged, $faculty, $department, $requester, $first_name, $last_name, $email, $username, $expiry) {
        
        $firstchar = substr($this->input->post('first_name'), 0, 1);
        $builtUsername = strtolower($firstchar) . strtolower($this->input->post('last_name'));
        
        //check user name is new
        $username = $this->config->item('ldapshortdomain') . $builtUsername;
        // specify the LDAP server to connect to
        $conn = ldap_connect($this->config->item('ldapserver')) or die("Oh no can't create LDAP connection");
        // bind to the LDAP server specified above
        if (!ldap_bind($conn, $this->config->item('ldapbindun'),"@".$this->config->item('ldapdomain'), $this->config->item('ldapbindpass')))

This code seems to be duplicated across your project.

            echo "Invalid credentials.";
        // Search for user in directory
        $cred = explode('\\', $username);
        list($domain, $user) = $cred;

The assignment to $domain is unused. Consider omitting it like so list($first,,$third).

        $result = ldap_search($conn, $this->config->item('ldapuserou'), "samaccountname=$user");
        // get entry data as array
        $info = ldap_count_entries($conn, $result);

        if ($info != 0) {
            $builtUsername = $builtUsername . "2";

$builtUsername is not used, you could remove the assignment.

        }

        function randomPassword() {

It is generally recommended to explicitly declare the visibility for methods.

            $alphabet = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890';
            $pass = array(); //remember to declare $pass as an array
            $alphaLength = strlen($alphabet) - 1; //put the length -1 in cache
            for ($i = 0; $i < 6; $i++) {
                $n = rand(0, $alphaLength);
                $pass[] = $alphabet[$n];
            }
            return implode($pass); //turn the array into a string
        }
        $generatedpassword = randomPassword();

        /** Returns UNIX timestamp and Active Directory timestamp for given date and time */
        //Format dd-mm-yyyy
        $dateFromForm = $this->input->post('expiry');

        //Format hh:mm:ss
        $timeFromForm = "00:00:00";
        $dateWithTime = $dateFromForm . " " . $timeFromForm;

$dateWithTime is not used, you could remove the assignment.

        function convertDateToUnix($input) {

It is generally recommended to explicitly declare the visibility for methods.

This function seems to be duplicated in your project.

The function convertDateToUnix() has been defined more than once; this definition is ignored, only the first definition in dashboard/application/controllers/computing-support/Temporary_account.php (L190-195) is considered.

            $format = 'd-m-Y H:i:s';
            $date = DateTime::createFromFormat($format, $input);
            $UNIXTimestamp = $date->getTimestamp();
            return $UNIXTimestamp;
        }

        function convertUnixtoWin($input) {

It is generally recommended to explicitly declare the visibility for methods.

The function convertUnixtoWin() has been defined more than once; this definition is ignored, only the first definition in dashboard/application/controllers/computing-support/Temporary_account.php (L197-199) is considered.

            return ($input + 11644473600) * 10000000;
        }

        //$UNIX = convertDateToUnix($dateWithTime);

56% of this comment could be valid code. Did you maybe forget this after debugging?

        //$WIN = convertUnixtoWin($UNIX);

56% of this comment could be valid code. Did you maybe forget this after debugging?

        $data = array(
            'logged' => $_SESSION['ldap']['full_name'],
            'logged_at' => date("Y-m-d H:i:s", time()),
            'requester' => $this->input->post('requester'),
            'faculty' => $this->input->post('faculty'),
            'department' => $this->input->post('department'),
            'first_name' => $this->input->post('first_name'),
            'last_name' => $this->input->post('last_name'),
            'email' => $this->input->post('email'),
            'username' => $this->input->post('username'),
            'expiry' => $this->input->post('expiry'),
            'password' => $generatedpassword,
            //'wintime' => $WIN,

67% of this comment could be valid code. Did you maybe forget this after debugging?

            //'unixtime' => $UNIX,

67% of this comment could be valid code. Did you maybe forget this after debugging?

            'status' => 0);

        return $this->db->insert('temporary_account', $data);
    }

    public function get_next_temp() {
        $query = $this->db->query("SELECT id FROM temporary_account ORDER BY id DESC LIMIT 1;");
        foreach ($query->result() as $row) {
            return $row->id + 1;
        }
    }

    function get_faculty() {

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

        $this->db->from('faculty');
        $this->db->order_by('faculty');
        $result = $this->db->get();
        $return = array();
        if ($result->num_rows() > 0) {
            foreach ($result->result_array() as $row) {
                $return[$row['cat_id']] = $row['faculty'];
            }
        }

        return $return;
    }

    function get_department() {

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

        if (!isset($_GET['cat'])) {
            $_GET['cat'] = '1';
        }
        $result = $this->db->get_where('facultysub', array('cat_id' => $_GET['cat']));
        $return = array();
        if ($result->num_rows() > 0) {
            foreach ($result->result_array() as $row) {
                $return[$row['cat_key']] = $row['subfaculty'];
            }
        }

        return $return;
    }
    
    public function get_pending() {
        $this->db->where('status', '0');
        $query = $this->db->get('temporary_account');
        return $query->result_array();
    }
    
    public function get_all() {
        $query = $this->db->get('temporary_account');
        return $query->result_array();
    }
    
    public function check_status() {
        $this->db->where('logged', $_SESSION['ldap']['full_name'])->or_where('requester', $_SESSION['ldap']['full_name']);
        $query = $this->db->get('temporary_account');
        return $query->result_array();
    }
    
    public function match_id_user($id) {
        $this->db->where('id', $id);
        $query = $this->db->get('temporary_account');
        return $query->result_array();
    }
    
    public function cancel($id) {
            
        $this->db->where('id', $id);
	$this->db->from('temporary_account');
        $data = array(
            'status' => '3',
        );
        return $this->db->update('temporary_account', $data);
    }
    
    public function reject($id) {

This method seems to be duplicated in your project.

            
        $this->db->where('id', $id);
	$this->db->from('temporary_account');
        $data = array(
            'status' => '2',
            'status_at' => date("Y-m-d H:i:s", time()),
            'status_by' => $_SESSION['ldap']['full_name'],
        );
        return $this->db->update('temporary_account', $data);
    }
    
    public function approve($id) {

This method seems to be duplicated in your project.

            
        $this->db->where('id', $id);
	$this->db->from('temporary_account');
        $data = array(
            'status' => '1',
            'status_at' => date("Y-m-d H:i:s", time()),
            'status_by' => $_SESSION['ldap']['full_name'],
        );
        
        return $this->db->update('temporary_account', $data);
    }
    
    public function created_account($id) {
            
        $this->db->where('id', $id);
	$this->db->from('temporary_account');
        $data = array(
            'created' => '1',
        );
        return $this->db->update('temporary_account', $data);
    }
    
    public function get_account($id) {
        $this->db->where('id', $id);
        $query = $this->db->get('temporary_account');
        return $query->result_array();
    }
}