chamilo/chamilo-lms

View on GitHub
public/main/lp/openoffice_document.class.php

Summary

Maintainability
A
0 mins
Test Coverage

The method convert_document() has an NPath complexity of 264. The configured NPath complexity threshold is 200.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

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

Remove error control operator '@' on line 126.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 164.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 124.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 163.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 125.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 240.
Open

    public function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = [];
        $originalPathInfo = pathinfo($originalPath);

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 251.
Open

    public function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = [];
        $originalPathInfo = pathinfo($originalPath);

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 250.
Open

    public function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = [];
        $originalPathInfo = pathinfo($originalPath);

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

Remove error control operator '@' on line 241.
Open

    public function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = [];
        $originalPathInfo = pathinfo($originalPath);

ErrorControlOperator

Error suppression should be avoided if possible as it doesn't just suppress the error, that you are trying to stop, but will also suppress errors that you didn't predict would ever occur. Consider changing error_reporting() level and/or setting up your own error handler.

Example

function foo($filePath) {
    $file = @fopen($filPath); // hides exceptions
    $key = @$array[$notExistingKey]; // assigns null to $key
}

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

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

        $client = new SoapClient(null, $options);

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 convert_document uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

        } else {
            $this->file_path = $this->created_dir.'/'.api_replace_dangerous_char($file['name']);
        }

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 convert_document uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

        } else {
            // get result from webservices
            $result = $this->_get_remote_ppt2lp_files($file, $size);
            $result = unserialize($result);
            // Save remote images to server

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 convert_document uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

            } else {
                $converter_path = api_get_path(SYS_PATH).'main/inc/lib/ppt2png';
                //$class_path = '-cp .:jodconverter-2.2.1.jar:jodconverter-cli-2.2.1.jar';
                $class_path = ' -Dfile.encoding=UTF-8 -cp .:jodconverter-2.2.2.jar:jodconverter-cli-2.2.2.jar';
                $cmd = 'cd '.$converter_path.' && java '.$class_path.' DokeosConverter';

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 _get_remote_ppt2lp_files uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

        } else {
            $uri = $ppt2lp_host;
        }

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 convertCopyDocument uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

            } else {
                $converterPath = api_get_path(SYS_PATH).'main/inc/lib/ppt2png';
                $classPath = ' -Dfile.encoding=UTF-8 -jar jodconverter-cli-2.2.2.jar';
                $cmd = 'cd '.$converterPath.' && java '.$classPath.' ';
            }

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 convertCopyDocument uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
Open

        } else {
            /*
             * @TODO Create method to use webservice
            // get result from webservices
            $result = $this->_get_remote_ppt2lp_files($file);

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 unused local variables such as '$shell'.
Open

            $shell = exec($cmd, $files, $return);

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$perm'.
Open

            $perm = api_get_setting('permissions_for_new_files');

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

Avoid unused local variables such as '$shell'.
Open

            $shell = exec($cmd, $files, $return);

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

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

Source https://phpmd.org/rules/unusedcode.html#unusedlocalvariable

The property $original_locale is not named in camelCase.
Open

abstract class OpenofficeDocument extends learnpath
{
    public $first_item = 0;
    public $original_charset = 'utf-8';
    public $original_locale = 'en_US.UTF-8';

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The property $slide_width is not named in camelCase.
Open

abstract class OpenofficeDocument extends learnpath
{
    public $first_item = 0;
    public $original_charset = 'utf-8';
    public $original_locale = 'en_US.UTF-8';

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The property $original_charset is not named in camelCase.
Open

abstract class OpenofficeDocument extends learnpath
{
    public $first_item = 0;
    public $original_charset = 'utf-8';
    public $original_locale = 'en_US.UTF-8';

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The parameter $user_id is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseParameterName

Since: 0.2

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

Example

class ClassName {
    public function doSomething($user_name) {
    }
}

Source

The property $first_item is not named in camelCase.
Open

abstract class OpenofficeDocument extends learnpath
{
    public $first_item = 0;
    public $original_charset = 'utf-8';
    public $original_locale = 'en_US.UTF-8';

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The parameter $course_code is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseParameterName

Since: 0.2

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

Example

class ClassName {
    public function doSomething($user_name) {
    }
}

Source

The parameter $action_after_conversion is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseParameterName

Since: 0.2

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

Example

class ClassName {
    public function doSomething($user_name) {
    }
}

Source

The parameter $resource_id is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseParameterName

Since: 0.2

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

Example

class ClassName {
    public function doSomething($user_name) {
    }
}

Source

The property $slide_height is not named in camelCase.
Open

abstract class OpenofficeDocument extends learnpath
{
    public $first_item = 0;
    public $original_charset = 'utf-8';
    public $original_locale = 'en_US.UTF-8';

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

Variable "created_dir" is not in valid camel caps format
Open

            $this->file_path = $this->created_dir.'/'.api_replace_dangerous_char($file['name']);

Variable "class_path" is not in valid camel caps format
Open

                $class_path = ' -Dfile.encoding=UTF-8 -cp .:jodconverter-2.2.2.jar:jodconverter-cli-2.2.2.jar';

Missing function doc comment
Open

    abstract public function add_docs_to_visio();

Missing parameter name
Open

     * @param    string    Course code

Variable "resource_id" is not in valid camel caps format
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)

Variable "file_name" is not in valid camel caps format
Open

        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);

Variable "file_path" is not in valid camel caps format
Open

            $this->file_path = $this->created_dir.'/'.api_replace_dangerous_char($file['name']);

Missing function doc comment
Open

    abstract public function add_command_parameters();

Variable "user_id" is not in valid camel caps format
Open

        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {

Variable "user_id" is not in valid camel caps format
Open

            parent::__construct($course_code, $resource_id, $user_id);

Variable "ppt2lp_host" is not in valid camel caps format
Open

        if ('localhost' === $ppt2lp_host) {

Variable "file_path" is not in valid camel caps format
Open

            move_uploaded_file($file['tmp_name'], $this->base_work_dir.'/'.$this->file_path);

Variable "original_locale" is not in valid camel caps format
Open

            $locale = $this->original_locale; // TODO: Improve it because we're not sure this locale is present everywhere.

Variable "img_data" is not in valid camel caps format
Open

                    @file_put_contents($image_path.'/'.$image, base64_decode($img_data));

Member variable "slide_width" is not in valid camel caps format
Open

    public $slide_width;

Spaces must be used for alignment; tabs are not allowed
Open

     * @param    string    Course code

Variable "resource_id" is not in valid camel caps format
Open

            parent::__construct($course_code, $resource_id, $user_id);

Variable "class_path" is not in valid camel caps format
Open

                $cmd = 'cd '.$converter_path.' && java '.$class_path.' DokeosConverter';

Variable "base_work_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.$this->created_dir, $dirMode);

Variable "file_path" is not in valid camel caps format
Open

            $this->file_path = $this->created_dir.api_replace_dangerous_char($file['name']);

Variable "ppt2lp_host" is not in valid camel caps format
Open

        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');

Variable "base_work_dir" is not in valid camel caps format
Open

                DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);

Variable "img_data" is not in valid camel caps format
Open

                foreach ($result['images'] as $image => $img_data) {

Variable "lp_id" is not in valid camel caps format
Open

            $this->lp_id = learnpath::add_lp($_course['id'], $this->file_name, '', 'guess', 'manual');

Variable "first_item" is not in valid camel caps format
Open

        return $this->first_item;

Variable "course_code" is not in valid camel caps format
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)

Variable "user_id" is not in valid camel caps format
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)

Variable "converter_path" is not in valid camel caps format
Open

                $class_path = $converter_path.';'.$converter_path.'/jodconverter-2.2.2.jar;'.$converter_path.'/jodconverter-cli-2.2.2.jar';

Variable "converter_path" is not in valid camel caps format
Open

                $converter_path = api_get_path(SYS_PATH).'main/inc/lib/ppt2png';

Variable "created_dir" is not in valid camel caps format
Open

                DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);

Variable "image_path" is not in valid camel caps format
Open

                    $image_path = $this->base_work_dir.$this->created_dir;

Variable "slide_height" is not in valid camel caps format
Open

                $this->slide_height = $h;

Member variable "original_locale" is not in valid camel caps format
Open

    public $original_locale = 'en_US.UTF-8';

Member variable "slide_height" is not in valid camel caps format
Open

    public $slide_height;

Spaces must be used for alignment; tabs are not allowed
Open

     * @param    int    User ID

Variable "slide_width" is not in valid camel caps format
Open

                $this->slide_width = $w;

Spaces must be used for alignment; tabs are not allowed
Open

 * @license    GNU/GPL

Variable "base_work_dir" is not in valid camel caps format
Open

            move_uploaded_file($file['tmp_name'], $this->base_work_dir.'/'.$this->file_path);

Variable "base_work_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.$this->file_path, $fileMode);

Abstract class name is not prefixed with "Abstract"
Open

abstract class OpenofficeDocument extends learnpath

Member variable "first_item" is not in valid camel caps format
Open

    public $first_item = 0;

Missing parameter name
Open

     * @param    int    Learnpath ID in DB

Variable "action_after_conversion" is not in valid camel caps format
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)

Variable "base_work_dir" is not in valid camel caps format
Open

        $this->base_work_dir = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';

Variable "created_dir" is not in valid camel caps format
Open

            $this->file_path = $this->created_dir.api_replace_dangerous_char($file['name']);

Variable "converter_path" is not in valid camel caps format
Open

                $class_path = $converter_path.';'.$converter_path.'/jodconverter-2.2.2.jar;'.$converter_path.'/jodconverter-cli-2.2.2.jar';

Variable "base_work_dir" is not in valid camel caps format
Open

            chmod($this->base_work_dir.$this->created_dir, api_get_permissions_for_new_directories());

Method name "OpenofficeDocument::add_docs_to_visio" is not in camel caps format
Open

    abstract public function add_docs_to_visio();

Spaces must be used for alignment; tabs are not allowed
Open

     * @param    int    Learnpath ID in DB

Variable "course_code" is not in valid camel caps format
Open

            parent::__construct($course_code, $resource_id, $user_id);

Variable "converter_path" is not in valid camel caps format
Open

                $converter_path = str_replace('/', '\\', api_get_path(SYS_PATH).'main/inc/lib/ppt2png');

Variable "class_path" is not in valid camel caps format
Open

                $class_path = $converter_path.';'.$converter_path.'/jodconverter-2.2.2.jar;'.$converter_path.'/jodconverter-cli-2.2.2.jar';

Member variable "original_charset" is not in valid camel caps format
Open

    public $original_charset = 'utf-8';

Variable "course_code" is not in valid camel caps format
Open

        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {

Method name "OpenofficeDocument::convert_document" is not in camel caps format
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)

Variable "converter_path" is not in valid camel caps format
Open

                $cmd = 'cd '.$converter_path.' && java '.$class_path.' DokeosConverter';

Variable "action_after_conversion" is not in valid camel caps format
Open

            switch ($action_after_conversion) {

Spaces must be used for alignment; tabs are not allowed
Open

 * @author    Eric Marguin <eric.marguin@dokeos.com>

Variable "file_name" is not in valid camel caps format
Open

        $result = $this->generate_lp_folder($_course, $this->file_name);

Variable "created_dir" is not in valid camel caps format
Open

        $this->created_dir = $result['dir'];

Variable "created_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.$this->created_dir, $dirMode);

Variable "created_dir" is not in valid camel caps format
Open

            chmod($this->base_work_dir.$this->created_dir, api_get_permissions_for_new_directories());

Variable "created_dir" is not in valid camel caps format
Open

                    $image_path = $this->base_work_dir.$this->created_dir;

Variable "base_work_dir" is not in valid camel caps format
Open

            chmod($this->base_work_dir, api_get_permissions_for_new_directories());

Missing function doc comment
Open

    abstract public function make_lp();

Spaces must be used for alignment; tabs are not allowed
Open

     * @param    int    User ID

Variable "resource_id" is not in valid camel caps format
Open

        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {

Variable "image_path" is not in valid camel caps format
Open

                    @file_put_contents($image_path.'/'.$image, base64_decode($img_data));

Variable "course_info" is not in valid camel caps format
Open

            $this->course_info = $_course;

Missing parameter name
Open

     * @param    int    User ID

Variable "created_dir" is not in valid camel caps format
Open

        if ('/' == substr($this->created_dir, -1, 1)) {

Variable "converter_path" is not in valid camel caps format
Open

                $class_path = $converter_path.';'.$converter_path.'/jodconverter-2.2.2.jar;'.$converter_path.'/jodconverter-cli-2.2.2.jar';

Variable "class_path" is not in valid camel caps format
Open

                $cmd = 'java -Dfile.encoding=UTF-8 -cp "'.$class_path.'" DokeosConverter';

Variable "base_work_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir, $dirMode);

Variable "base_work_dir" is not in valid camel caps format
Open

                    $image_path = $this->base_work_dir.$this->created_dir;

Variable "image_path" is not in valid camel caps format
Open

                    @chmod($image_path.'/'.$image, $fileMode);

Method name "OpenofficeDocument::make_lp" is not in camel caps format
Open

    abstract public function make_lp();

Variable "file_path" is not in valid camel caps format
Open

        $this->file_path = $originalPathInfo['basename'];

Variable "file_name" is not in valid camel caps format
Open

        $file_name = $file['name'];

Variable "original_locale" is not in valid camel caps format
Open

            $locale = $this->original_locale; // TODO: Improve it because we're not sure this locale is present everywhere.

Variable "base_work_dir" is not in valid camel caps format
Open

        if (file_exists($this->base_work_dir.'/'.$this->created_dir)) {

Variable "ppt2lp_host" is not in valid camel caps format
Open

        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');

Variable "file_data" is not in valid camel caps format
Open

        $file_data = base64_encode(file_get_contents($file['tmp_name']));

Variable "base_work_dir" is not in valid camel caps format
Open

            $cmd .= ' "'.$this->base_work_dir.'/'.$this->file_path.'"  "'.$this->base_work_dir.'/'.$this->created_dir.'"';

Variable "created_dir" is not in valid camel caps format
Open

            $cmd .= ' "'.$this->base_work_dir.'/'.$this->file_path.'"  "'.$this->base_work_dir.'/'.$this->created_dir.'"';

Variable "created_dir" is not in valid camel caps format
Open

                '/'.$this->created_dir,

Variable "created_dir" is not in valid camel caps format
Open

                filesize($this->base_work_dir.'/'.$this->created_dir),

Variable "base_work_dir" is not in valid camel caps format
Open

            chmod($this->base_work_dir, $permissionFolder);

Variable "secret_key" is not in valid camel caps format
Open

            'secret_key' => $secret_key,

Variable "base_work_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.'/'.$this->file_path, $permissionFile);

Method name "OpenofficeDocument::_get_remote_ppt2lp_files" is not in camel caps format
Open

    private function _get_remote_ppt2lp_files($file, $size = null)

Spaces must be used for alignment; tabs are not allowed
Open

     * @param    string    Course code

Spaces must be used for alignment; tabs are not allowed
Open

     * @param    int    Learnpath ID in DB

Variable "file_path" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.$this->file_path, $fileMode);

Variable "file_name" is not in valid camel caps format
Open

            $this->lp_id = learnpath::add_lp($_course['id'], $this->file_name, '', 'guess', 'manual');

Method name "OpenofficeDocument::add_command_parameters" is not in camel caps format
Open

    abstract public function add_command_parameters();

Variable "created_dir" is not in valid camel caps format
Open

        $this->created_dir = $convertedPathInfo['basename'];

Variable "created_dir" is not in valid camel caps format
Open

            @chown($this->base_work_dir.'/'.$this->created_dir, 'www-data');

Method name "_get_remote_ppt2lp_files" should not be prefixed with an underscore to indicate visibility
Open

    private function _get_remote_ppt2lp_files($file, $size = null)

Variable "base_work_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir, $permissionFolder);

Variable "file_data" is not in valid camel caps format
Open

            'file_data' => $file_data,

Variable "base_work_dir" is not in valid camel caps format
Open

        $this->base_work_dir = $originalPathInfo['dirname'];

Variable "ppt2lp_host" is not in valid camel caps format
Open

        $result = preg_match('/^([a-zA-Z0-9]*):\/\/([^\/]*)\//', $ppt2lp_host, $matches);

Variable "base_work_dir" is not in valid camel caps format
Open

        if (file_exists($this->base_work_dir.'/'.$this->created_dir)) {

Variable "created_dir" is not in valid camel caps format
Open

                DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);

Variable "base_work_dir" is not in valid camel caps format
Open

            $cmd .= ' "'.$this->base_work_dir.'/'.$this->file_path.'"  "'.$this->base_work_dir.'/'.$this->created_dir.'"';

Variable "created_dir" is not in valid camel caps format
Open

        if (file_exists($this->base_work_dir.'/'.$this->created_dir)) {

Variable "base_work_dir" is not in valid camel caps format
Open

                filesize($this->base_work_dir.'/'.$this->created_dir),

Variable "ppt2lp_host" is not in valid camel caps format
Open

            'location' => $ppt2lp_host,

Variable "ppt2lp_host" is not in valid camel caps format
Open

        if ('https' === substr($ppt2lp_host, 0, 5)) {

Variable "file_path" is not in valid camel caps format
Open

            $cmd .= ' "'.$this->base_work_dir.'/'.$this->file_path.'"  "'.$this->base_work_dir.'/'.$this->created_dir.'"';

Variable "base_work_dir" is not in valid camel caps format
Open

            @chown($this->base_work_dir.'/'.$this->created_dir, 'www-data');

Variable "base_work_dir" is not in valid camel caps format
Open

                DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);

Variable "file_name" is not in valid camel caps format
Open

            'file_name' => $file_name,

Variable "created_dir" is not in valid camel caps format
Open

        if (file_exists($this->base_work_dir.'/'.$this->created_dir)) {

Variable "base_work_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.'/'.$this->created_dir, $permissionFile);

Variable "created_dir" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.'/'.$this->created_dir, $permissionFile);

Variable "file_path" is not in valid camel caps format
Open

            @chmod($this->base_work_dir.'/'.$this->file_path, $permissionFile);

Variable "ppt2lp_host" is not in valid camel caps format
Open

            $uri = $ppt2lp_host;

Variable "secret_key" is not in valid camel caps format
Open

        $secret_key = sha1(api_get_setting('service_ppt2lp', 'ftp_password'));

The variable $resource_id is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $image_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $user_id is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $image_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $converter_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $img_data is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_code is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $resource_id is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $user_id is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $ppt2lp_host is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $ppt2lp_host is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $converter_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $converter_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $class_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $class_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $img_data is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $course_code is not named in camelCase.
Open

    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $converter_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $class_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $converter_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $image_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $action_after_conversion is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $converter_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $class_path is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = [];
        $originalPathInfo = pathinfo($originalPath);

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $file_data is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = [];
        $originalPathInfo = pathinfo($originalPath);

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $ppt2lp_host is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $_course is not named in camelCase.
Open

    public function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = [];
        $originalPathInfo = pathinfo($originalPath);

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $ppt2lp_host is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $secret_key is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $secret_key is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $file_name is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $file_name is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $ppt2lp_host is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $ppt2lp_host is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $ppt2lp_host is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The variable $file_data is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

CamelCaseVariableName

Since: 0.2

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

Example

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

Source

The method make_lp is not named in camelCase.
Open

    abstract public function make_lp();

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 convert_document is not named in camelCase.
Open

    public function convert_document($file, $action_after_conversion = 'make_lp', $size = null)
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory

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_docs_to_visio is not named in camelCase.
Open

    abstract public function add_docs_to_visio();

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_command_parameters is not named in camelCase.
Open

    abstract public function add_command_parameters();

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 _get_remote_ppt2lp_files is not named in camelCase.
Open

    private function _get_remote_ppt2lp_files($file, $size = null)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)

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