felicity-iiith / felicity18-website

src/app/controllers/ajax.php

<?php

class ajax extends Controller {

    function __construct() {

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

        $this->load_library("http_lib", "http");
        $this->load_library("auth_lib", "auth");

        $this->load_model("jugaad_model");
        $this->load_model("perms_model");
        $this->load_model("auth_model");

        $this->user = $this->auth->get_user() ?: "";

The property auth does not exist on ajax. Did you maybe forget to declare it?

The property user does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
     * @param string $param Get GET parameter called $param
     * @return mixed GET parameter or false if not found
     */
    private function get_param($param) {
        if (isset($_GET[$param])) {
            return $_GET[$param];
        }
        return false;
    }

    function latest_version_id() {

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

        $file_id = $this->get_param("file_id");

        $file = $this->jugaad_model->get_file($file_id);

The property jugaad_model does not exist on ajax. Did you maybe forget to declare it?

        if ($file === false || $file["type"] != "file") {
            $this->http->response_code(404);

The property http does not exist on ajax. Did you maybe forget to declare it?

        }

        $user_can = $this->perms_model->get_permissions($file_id, $this->user);

The property perms_model does not exist on ajax. Did you maybe forget to declare it?

        if (!$user_can['read_file']) {
            $this->http->response_code(403);
        }

        $version_id = $this->jugaad_model->get_latest_version_id($file_id);

        echo $version_id;
    }

}