OpenofficeDocument::_get_remote_ppt2lp_files() - Code Metrics - Inspection of "Improvements to remote PPT2LP SOAP call to compres..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 1.10.x ( 4a4ede...319c16 )

by Yannick

created 2016-11-21 22:36 UTC

OpenofficeDocument::_get_remote_ppt2lp_files() A

↳ Parent: OpenofficeDocument

Complexity

Conditions	4
Paths	8

Size

Total Lines	55
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	37
nc	8
nop	1
dl	0
loc	55
rs	9.078
c	0
b	0
f	0

How to fix Long Method

<?php

/* For licensing terms, see /license.txt */

/**
 * Defines the OpenofficeDocument class, which is meant as a mother class
 * to help in the conversion of Office documents to learning paths
 * @package chamilo.learnpath
 * @author	Eric Marguin <[email protected]>
 * @author Julio Montoya
 * @license	GNU/GPL
 */

/**
 * Defines the "OpenofficeDocument" child of class "learnpath"
 */
abstract class OpenofficeDocument extends learnpath
{

    public $first_item = 0;
    public $original_charset = 'utf-8';
    public $original_locale = 'en_US.UTF-8';

    /**
     * Class constructor. Based on the parent constructor.
     * @param	string	Course code
     * @param	integer	Learnpath ID in DB
     * @param	integer	User ID
     */
    public function __construct($course_code = null, $resource_id = null, $user_id = null)
    {
        if ($this->debug > 0) {
            error_log('In OpenofficeDocument::OpenofficeDocument()', 0);
        }
        if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
            parent::__construct($course_code, $resource_id, $user_id);
        }
    }

    /**
     * @param string $file
     * @param string $action_after_conversion
     * @return bool|int
     */
    public function convert_document($file, $action_after_conversion = 'make_lp')
    {
        $_course = api_get_course_info();
        $this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
        // Create the directory
        $result = $this->generate_lp_folder($_course, $this->file_name);

         // Create the directory
        $this->base_work_dir = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
        ///learning_path/ppt_dirname directory
        $this->created_dir = substr($result['dir'], 0, strlen($result['dir']) -1);
        $this->file_path = $this->created_dir.'/'.api_replace_dangerous_char($file['name']);

        //var_dump($this->file_name, $this->file_path, $this->base_work_dir, $this->created_dir);

        /*
         * Original code
        global $_course, $_user, $_configuration;

        $this->file_name = (strrpos($file['name'], '.') > 0 ? substr($file['name'], 0, strrpos($file['name'], '.')) : $file['name']);
        $this->file_name = api_replace_dangerous_char($this->file_name, 'strict');
        $this->file_name = strtolower($this->file_name);

        $visio_dir = ($action_after_conversion == 'add_docs_to_visio') ? VIDEOCONF_UPLOAD_PATH : '';

        $this->file_path = $visio_dir.'/'.$this->file_name.'.'.pathinfo($file['name'], PATHINFO_EXTENSION);

        $dir_name = $visio_dir.'/'.$this->file_name;


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

        $this->created_dir = create_unexisting_directory($_course, $_user['user_id'], api_get_session_id(), 0, 0, $this->base_work_dir, $dir_name);

            var_dump($this->file_name, $this->file_path, $this->base_work_dir, $this->created_dir);

        */

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

        if ($ppt2lp_host == 'localhost') {
            move_uploaded_file($file['tmp_name'], $this->base_work_dir.'/'.$this->file_path);
            //var_dump( $this->base_work_dir.$this->created_dir.$this->file_path);
            $perm = api_get_setting('permissions_for_new_files');

            if (IS_WINDOWS_OS) { // IS_WINDOWS_OS has been defined in main_api.lib.php
                $converter_path = str_replace('/', '\\', api_get_path(SYS_PATH) . 'main/inc/lib/ppt2png');
                $class_path = $converter_path . ';' . $converter_path . '/jodconverter-2.2.2.jar;' . $converter_path . '/jodconverter-cli-2.2.2.jar';
                //$cmd = 'java -cp "'.$class_path.'" DokeosConverter';
                $cmd = 'java -Dfile.encoding=UTF-8 -cp "' . $class_path . '" DokeosConverter';
            } 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';
            }

            $cmd .= ' -p ' . api_get_setting('service_ppt2lp', 'port');
            // Call to the function implemented by child.
            $cmd .= $this->add_command_parameters();
            // To allow openoffice to manipulate docs.
            @chmod($this->base_work_dir, 0777);
            @chmod($this->base_work_dir.$this->created_dir, 0777);
            @chmod($this->base_work_dir.$this->file_path, 0777);

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

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

            if ($return != 0) { // If the java application returns an error code.

                switch ($return) {
                    // Can't connect to openoffice.
                    case 1: $this->error = get_lang('CannotConnectToOpenOffice');
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                        break;
                    // Conversion failed in openoffice.
                    case 2: $this->error = get_lang('OogieConversionFailed');
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                        break;
                    // Conversion can't be launch because command failed.
                    case 255: $this->error = get_lang('OogieUnknownError');
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                        break;
                }
                DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);
                return false;
            }
        } else {
            // get result from webservices
            $result = $this->_get_remote_ppt2lp_files($file);
            $result = unserialize($result);
            // Save remote images to server
            chmod($this->base_work_dir.$this->created_dir, api_get_permissions_for_new_directories());
            if (!empty($result['images'])) {
                foreach ($result['images'] as $image => $img_data) {
                    $image_path = $this->base_work_dir.$this->created_dir;
                    @file_put_contents($image_path . '/' . $image, base64_decode($img_data));
                    @chmod($image_path . '/' . $image, 0777);
                }
            }

            // files info
            $files = $result['files'];
        }

        if (!empty($files)) {
            // Create lp
            $this->lp_id = learnpath::add_lp($_course['id'], $this->file_name, '', 'guess', 'manual');
            // make sure we have a course code available for later
            $this->cc = $_course['id'];

            // Call to the function implemented by child following action_after_conversion parameter.
            switch ($action_after_conversion) {
                case 'make_lp':
                    $this->make_lp($files);
                    break;
                case 'add_docs_to_visio':
                    $this->add_docs_to_visio($files);
                    break;
            }
            chmod($this->base_work_dir, api_get_permissions_for_new_directories());
        }
        return $this->first_item;
    }

    /**
     * Get images files from remote host (with webservices)
     * @param   array current ppt file
     * @return  array images files
     */
    private function _get_remote_ppt2lp_files($file)
    {
        // host
        $ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
        // SOAP URI (just the host)
        $matches = array();
        $uri = '';
        $result = preg_match('/^([a-zA-Z0-9]*):\/\/([^\/]*)\//', $ppt2lp_host, $matches);
        if ($result) {
            $uri = $matches[1] . '://' . $matches[2] . '/';
        } else {
            $uri = $ppt2lp_host;
        }
        // secret key
        $secret_key = sha1(api_get_setting('service_ppt2lp', 'ftp_password'));

        // client
        $options = array(
            'location' => $ppt2lp_host,
            'uri' => $uri,
            'trace' => 1,
            'exceptions' => true,
            'cache_wsdl' => WSDL_CACHE_NONE,
            'keep_alive' => false,
            'features' => SOAP_SINGLE_ELEMENT_ARRAYS,
            'compression' => SOAP_COMPRESSION_ACCEPT | SOAP_COMPRESSION_GZIP | 9,
        );
        if (substr($ppt2lp_host, 0, 5) === 'https') {
            $options['ssl_method'] =  SOAP_SSL_METHOD_TLS;
        }
        $client = new SoapClient(null, $options);
        $result = '';

        $file_data = base64_encode(file_get_contents($file['tmp_name']));
        $file_name = $file['name'];
        $service_ppt2lp_size = api_get_setting('service_ppt2lp', 'size');

        $params = array(
            'secret_key' => $secret_key,
            'file_data' => $file_data,
            'file_name' => $file_name,
            'service_ppt2lp_size' => $service_ppt2lp_size,
        );

        try {
            //error_log('['.time().'] Calling wsConvertPpt webservice on ' . $ppt2lp_host);
            $result = $client->__call('wsConvertPpt', array('pptData' => $params));
        } catch (Exception $e) {
            error_log('['.time().'] Chamilo SOAP call error: ' . $e->getMessage());
        }
        // Make sure we destroy the SOAP client as it may generate SSL connection
        // binding issue (if using SSL)
        unset($client);
        return $result;
    }

    abstract function make_lp();

    abstract function add_docs_to_visio();

    abstract function add_command_parameters();

    /**
     * Used to convert copied from document
     * @param string $originalPath
     * @param string $convertedPath
     * @param string $convertedTitle
     * @return bool
     */
    function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
    {
        $_course = api_get_course_info();
        $ids = array();
        $originalPathInfo = pathinfo($originalPath);
        $convertedPathInfo = pathinfo($convertedPath);
        $this->base_work_dir = $originalPathInfo['dirname'];
        $this->file_path = $originalPathInfo['basename'];
        $this->created_dir = $convertedPathInfo['basename'];
        $ppt2lpHost = api_get_setting('service_ppt2lp', 'host');
        $permissionFile = api_get_permissions_for_new_files();
        $permissionFolder = api_get_permissions_for_new_directories();
        if (file_exists($this->base_work_dir . '/' . $this->created_dir)) {

            return $ids;
        }

        if ($ppt2lpHost == 'localhost') {
            if (IS_WINDOWS_OS) { // IS_WINDOWS_OS has been defined in main_api.lib.php
                $converterPath = str_replace('/', '\\', api_get_path(SYS_PATH) . 'main/inc/lib/ppt2png');
                $classPath = $converterPath . ';' . $converterPath . '/jodconverter-2.2.2.jar;' . $converterPath . '/jodconverter-cli-2.2.2.jar';
                $cmd = 'java -Dfile.encoding=UTF-8 -jar "' . $classPath . '/jodconverter-2.2.2.jar"';
            } 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 . ' ';
            }

            $cmd .= ' -p ' . api_get_setting('service_ppt2lp', 'port');
            // Call to the function implemented by child.
            $cmd .= ' "' . $this->base_work_dir . '/' . $this->file_path . '"  "' . $this->base_work_dir . '/' . $this->created_dir . '"';
            // To allow openoffice to manipulate docs.
            @chmod($this->base_work_dir, $permissionFolder);
            @chmod($this->base_work_dir . '/' . $this->file_path, $permissionFile);

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

            $files = array();
            $return = 0;
            $shell = exec($cmd, $files, $return);
            // TODO: Chown is not working, root keep user privileges, should be www-data
            @chown($this->base_work_dir . '/' . $this->created_dir, 'www-data');
            @chmod($this->base_work_dir . '/' . $this->created_dir, $permissionFile);

            if ($return != 0) { // If the java application returns an error code.

                switch ($return) {
                    // Can't connect to openoffice.
                    case 1: $this->error = get_lang('CannotConnectToOpenOffice');
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                        break;
                    // Conversion failed in openoffice.
                    case 2: $this->error = get_lang('OogieConversionFailed');
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                        break;
                    // Conversion can't be launch because command failed.
                    case 255: $this->error = get_lang('OogieUnknownError');
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                        break;
                }
                DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);
                return false;
            }
        } else {
if (rand(1, 6) > 3) {
print "Check failed";
} else {
    //print "Check succeeded";
}
            /*
             * @TODO Create method to use webservice
            // get result from webservices
            $result = $this->_get_remote_ppt2lp_files($file);
            $result = unserialize(base64_decode($result));

            // Save remote images to server
            chmod($this->base_work_dir.$this->created_dir, api_get_permissions_for_new_directories());
            if (!empty($result['images'])) {
                foreach ($result['images'] as $image => $img_data) {
                    $image_path = $this->base_work_dir.$this->created_dir;
                    @file_put_contents($image_path . '/' . $image, base64_decode($img_data));
                    @chmod($image_path . '/' . $image, 0777);
                }
            }

            // files info
            $files = $result['files'];
            */
        }

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

            // Register Files to Document tool
            $ids[] = add_document(
                $_course,
                '/' . $this->created_dir,
                'file',
                filesize($this->base_work_dir . '/' . $this->created_dir),
                $convertedTitle,
                sprintf(
                    get_lang('FileConvertedFromXToY'),
                    strtoupper($originalPathInfo['extension']),
                    strtoupper($convertedPathInfo['extension'])
                ),
                0,
                true,
                null,
                api_get_session_id()
            );
            chmod($this->base_work_dir, $permissionFolder);
        }

        return $ids;
    }
}


1		<?php
2
3		/* For licensing terms, see /license.txt */
4
5		/**
6		* Defines the OpenofficeDocument class, which is meant as a mother class
7		* to help in the conversion of Office documents to learning paths
8		* @package chamilo.learnpath
9		* @author Eric Marguin <[email protected]>
10		* @author Julio Montoya
11		* @license GNU/GPL
12		*/
13
14		/**
15		* Defines the "OpenofficeDocument" child of class "learnpath"
16		*/
17		abstract class OpenofficeDocument extends learnpath
18		{
19
20		public $first_item = 0;
21		public $original_charset = 'utf-8';
22		public $original_locale = 'en_US.UTF-8';
23
24		/**
25		* Class constructor. Based on the parent constructor.
26		* @param string Course code
27		* @param integer Learnpath ID in DB
28		* @param integer User ID
29		*/
30	View Code Duplication	public function __construct($course_code = null, $resource_id = null, $user_id = null)
31		{
32		if ($this->debug > 0) {
33		error_log('In OpenofficeDocument::OpenofficeDocument()', 0);
34		}
35		if (!empty($course_code) && !empty($resource_id) && !empty($user_id)) {
36		parent::__construct($course_code, $resource_id, $user_id);
37		}
38		}
39
40		/**
41		* @param string $file
42		* @param string $action_after_conversion
43		* @return bool\|int
44		*/
45		public function convert_document($file, $action_after_conversion = 'make_lp')
46		{
47		$_course = api_get_course_info();
48		$this->file_name = pathinfo($file['name'], PATHINFO_FILENAME);
49		// Create the directory
50		$result = $this->generate_lp_folder($_course, $this->file_name);
51
52		// Create the directory
53		$this->base_work_dir = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
54		///learning_path/ppt_dirname directory
55		$this->created_dir = substr($result['dir'], 0, strlen($result['dir']) -1);
56		$this->file_path = $this->created_dir.'/'.api_replace_dangerous_char($file['name']);
57
58		//var_dump($this->file_name, $this->file_path, $this->base_work_dir, $this->created_dir);
59
60		/*
61		* Original code
62		global $_course, $_user, $_configuration;
63
64		$this->file_name = (strrpos($file['name'], '.') > 0 ? substr($file['name'], 0, strrpos($file['name'], '.')) : $file['name']);
65		$this->file_name = api_replace_dangerous_char($this->file_name, 'strict');
66		$this->file_name = strtolower($this->file_name);
67
68		$visio_dir = ($action_after_conversion == 'add_docs_to_visio') ? VIDEOCONF_UPLOAD_PATH : '';
69
70		$this->file_path = $visio_dir.'/'.$this->file_name.'.'.pathinfo($file['name'], PATHINFO_EXTENSION);
71
72		$dir_name = $visio_dir.'/'.$this->file_name;
73
74
75		// Create the directory.
76		$this->base_work_dir = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
77
78		$this->created_dir = create_unexisting_directory($_course, $_user['user_id'], api_get_session_id(), 0, 0, $this->base_work_dir, $dir_name);
79
80		var_dump($this->file_name, $this->file_path, $this->base_work_dir, $this->created_dir);
81
82		*/
83
84		$ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
85
86		if ($ppt2lp_host == 'localhost') {
87		move_uploaded_file($file['tmp_name'], $this->base_work_dir.'/'.$this->file_path);
88		//var_dump( $this->base_work_dir.$this->created_dir.$this->file_path);
89		$perm = api_get_setting('permissions_for_new_files');
90
91	View Code Duplication	if (IS_WINDOWS_OS) { // IS_WINDOWS_OS has been defined in main_api.lib.php
92		$converter_path = str_replace('/', '\\', api_get_path(SYS_PATH) . 'main/inc/lib/ppt2png');
93		$class_path = $converter_path . ';' . $converter_path . '/jodconverter-2.2.2.jar;' . $converter_path . '/jodconverter-cli-2.2.2.jar';
94		//$cmd = 'java -cp "'.$class_path.'" DokeosConverter';
95		$cmd = 'java -Dfile.encoding=UTF-8 -cp "' . $class_path . '" DokeosConverter';
96		} else {
97		$converter_path = api_get_path(SYS_PATH) . 'main/inc/lib/ppt2png';
98		//$class_path = '-cp .:jodconverter-2.2.1.jar:jodconverter-cli-2.2.1.jar';
99		$class_path = ' -Dfile.encoding=UTF-8 -cp .:jodconverter-2.2.2.jar:jodconverter-cli-2.2.2.jar';
100		$cmd = 'cd ' . $converter_path . ' && java ' . $class_path . ' DokeosConverter';
101		}
102
103		$cmd .= ' -p ' . api_get_setting('service_ppt2lp', 'port');
104		// Call to the function implemented by child.
105		$cmd .= $this->add_command_parameters();
106		// To allow openoffice to manipulate docs.
107		@chmod($this->base_work_dir, 0777);
108		@chmod($this->base_work_dir.$this->created_dir, 0777);
109		@chmod($this->base_work_dir.$this->file_path, 0777);
110
111		$locale = $this->original_locale; // TODO: Improve it because we're not sure this locale is present everywhere.
112		putenv('LC_ALL=' . $locale);
113
114		$files = array();
115		$return = 0;
116		$shell = exec($cmd, $files, $return);
117
118	View Code Duplication	if ($return != 0) { // If the java application returns an error code.
		0 ignored issues – show Bug introduced 2016-02-26 17:07 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `$return` of type `integer\|null` to `0`; this is ambiguous as not only `0 == 0` is true, but `null == 0` is true, too. Consider using a strict comparison `===`. Loading history...
119		switch ($return) {
120		// Can't connect to openoffice.
121		case 1: $this->error = get_lang('CannotConnectToOpenOffice');
		0 ignored issues – show Coding Style introduced 2016-02-26 19:13 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
122		break;
123		// Conversion failed in openoffice.
124		case 2: $this->error = get_lang('OogieConversionFailed');
		0 ignored issues – show Coding Style introduced 2016-02-26 19:13 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
125		break;
126		// Conversion can't be launch because command failed.
127		case 255: $this->error = get_lang('OogieUnknownError');
		0 ignored issues – show Coding Style introduced 2016-02-26 19:13 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
128		break;
129		}
130		DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);
131		return false;
132		}
133		} else {
134		// get result from webservices
135		$result = $this->_get_remote_ppt2lp_files($file);
136		$result = unserialize($result);
137		// Save remote images to server
138		chmod($this->base_work_dir.$this->created_dir, api_get_permissions_for_new_directories());
139		if (!empty($result['images'])) {
140		foreach ($result['images'] as $image => $img_data) {
141		$image_path = $this->base_work_dir.$this->created_dir;
142		@file_put_contents($image_path . '/' . $image, base64_decode($img_data));
143		@chmod($image_path . '/' . $image, 0777);
144		}
145		}
146
147		// files info
148		$files = $result['files'];
149		}
150
151		if (!empty($files)) {
152		// Create lp
153		$this->lp_id = learnpath::add_lp($_course['id'], $this->file_name, '', 'guess', 'manual');
154		// make sure we have a course code available for later
155		$this->cc = $_course['id'];
156
157		// Call to the function implemented by child following action_after_conversion parameter.
158		switch ($action_after_conversion) {
159		case 'make_lp':
160		$this->make_lp($files);
161		break;
162		case 'add_docs_to_visio':
163		$this->add_docs_to_visio($files);
164		break;
165		}
166		chmod($this->base_work_dir, api_get_permissions_for_new_directories());
167		}
168		return $this->first_item;
169		}
170
171		/**
172		* Get images files from remote host (with webservices)
173		* @param array current ppt file
174		* @return array images files
175		*/
176		private function _get_remote_ppt2lp_files($file)
177		{
178		// host
179		$ppt2lp_host = api_get_setting('service_ppt2lp', 'host');
180		// SOAP URI (just the host)
181		$matches = array();
182		$uri = '';
183		$result = preg_match('/^([a-zA-Z0-9]):\/\/([^\/])\//', $ppt2lp_host, $matches);
184		if ($result) {
185		$uri = $matches[1] . '://' . $matches[2] . '/';
186		} else {
187		$uri = $ppt2lp_host;
188		}
189		// secret key
190		$secret_key = sha1(api_get_setting('service_ppt2lp', 'ftp_password'));
191
192		// client
193		$options = array(
194		'location' => $ppt2lp_host,
195		'uri' => $uri,
196		'trace' => 1,
197		'exceptions' => true,
198		'cache_wsdl' => WSDL_CACHE_NONE,
199		'keep_alive' => false,
200		'features' => SOAP_SINGLE_ELEMENT_ARRAYS,
201		'compression' => SOAP_COMPRESSION_ACCEPT \| SOAP_COMPRESSION_GZIP \| 9,
202		);
203		if (substr($ppt2lp_host, 0, 5) === 'https') {
204		$options['ssl_method'] = SOAP_SSL_METHOD_TLS;
205		}
206		$client = new SoapClient(null, $options);
207		$result = '';
208
209		$file_data = base64_encode(file_get_contents($file['tmp_name']));
210		$file_name = $file['name'];
211		$service_ppt2lp_size = api_get_setting('service_ppt2lp', 'size');
212
213		$params = array(
214		'secret_key' => $secret_key,
215		'file_data' => $file_data,
216		'file_name' => $file_name,
217		'service_ppt2lp_size' => $service_ppt2lp_size,
218		);
219
220		try {
221		//error_log('['.time().'] Calling wsConvertPpt webservice on ' . $ppt2lp_host);
222		$result = $client->__call('wsConvertPpt', array('pptData' => $params));
223		} catch (Exception $e) {
224		error_log('['.time().'] Chamilo SOAP call error: ' . $e->getMessage());
225		}
226		// Make sure we destroy the SOAP client as it may generate SSL connection
227		// binding issue (if using SSL)
228		unset($client);
229		return $result;
230		}
231
232		abstract function make_lp();
233
234		abstract function add_docs_to_visio();
235
236		abstract function add_command_parameters();
237
238		/**
239		* Used to convert copied from document
240		* @param string $originalPath
241		* @param string $convertedPath
242		* @param string $convertedTitle
243		* @return bool
244		*/
245		function convertCopyDocument($originalPath, $convertedPath, $convertedTitle)
246		{
247		$_course = api_get_course_info();
248		$ids = array();
249		$originalPathInfo = pathinfo($originalPath);
250		$convertedPathInfo = pathinfo($convertedPath);
251		$this->base_work_dir = $originalPathInfo['dirname'];
252		$this->file_path = $originalPathInfo['basename'];
253		$this->created_dir = $convertedPathInfo['basename'];
254		$ppt2lpHost = api_get_setting('service_ppt2lp', 'host');
255		$permissionFile = api_get_permissions_for_new_files();
256		$permissionFolder = api_get_permissions_for_new_directories();
257		if (file_exists($this->base_work_dir . '/' . $this->created_dir)) {
258
259		return $ids;
260		}
261
262		if ($ppt2lpHost == 'localhost') {
263	View Code Duplication	if (IS_WINDOWS_OS) { // IS_WINDOWS_OS has been defined in main_api.lib.php
264		$converterPath = str_replace('/', '\\', api_get_path(SYS_PATH) . 'main/inc/lib/ppt2png');
265		$classPath = $converterPath . ';' . $converterPath . '/jodconverter-2.2.2.jar;' . $converterPath . '/jodconverter-cli-2.2.2.jar';
266		$cmd = 'java -Dfile.encoding=UTF-8 -jar "' . $classPath . '/jodconverter-2.2.2.jar"';
267		} else {
268		$converterPath = api_get_path(SYS_PATH) . 'main/inc/lib/ppt2png';
269		$classPath = ' -Dfile.encoding=UTF-8 -jar jodconverter-cli-2.2.2.jar';
270		$cmd = 'cd ' . $converterPath . ' && java ' . $classPath . ' ';
271		}
272
273		$cmd .= ' -p ' . api_get_setting('service_ppt2lp', 'port');
274		// Call to the function implemented by child.
275		$cmd .= ' "' . $this->base_work_dir . '/' . $this->file_path . '" "' . $this->base_work_dir . '/' . $this->created_dir . '"';
276		// To allow openoffice to manipulate docs.
277		@chmod($this->base_work_dir, $permissionFolder);
278		@chmod($this->base_work_dir . '/' . $this->file_path, $permissionFile);
279
280		$locale = $this->original_locale; // TODO: Improve it because we're not sure this locale is present everywhere.
281		putenv('LC_ALL=' . $locale);
282
283		$files = array();
284		$return = 0;
285		$shell = exec($cmd, $files, $return);
286		// TODO: Chown is not working, root keep user privileges, should be www-data
287		@chown($this->base_work_dir . '/' . $this->created_dir, 'www-data');
288		@chmod($this->base_work_dir . '/' . $this->created_dir, $permissionFile);
289
290	View Code Duplication	if ($return != 0) { // If the java application returns an error code.
		0 ignored issues – show Bug introduced 2016-02-26 17:07 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `$return` of type `integer\|null` to `0`; this is ambiguous as not only `0 == 0` is true, but `null == 0` is true, too. Consider using a strict comparison `===`. Loading history...
291		switch ($return) {
292		// Can't connect to openoffice.
293		case 1: $this->error = get_lang('CannotConnectToOpenOffice');
		0 ignored issues – show Coding Style introduced 2016-02-26 19:13 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
294		break;
295		// Conversion failed in openoffice.
296		case 2: $this->error = get_lang('OogieConversionFailed');
		0 ignored issues – show Coding Style introduced 2016-02-26 19:13 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
297		break;
298		// Conversion can't be launch because command failed.
299		case 255: $this->error = get_lang('OogieUnknownError');
		0 ignored issues – show Coding Style introduced 2016-02-26 19:13 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
300		break;
301		}
302		DocumentManager::delete_document($_course, $this->created_dir, $this->base_work_dir);
303		return false;
304		}
305		} else {
		0 ignored issues – show Unused Code introduced 2016-02-26 17:08 UTC by Report Bug Copy Issue Report This `else` statement is empty and can be removed. This check looks for the `else` branches of `if` statements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete. These `else` branches can be removed. if (rand(1, 6) > 3) { print "Check failed"; } else { //print "Check succeeded"; } could be turned into if (rand(1, 6) > 3) { print "Check failed"; } This is much more concise to read. Loading history...
306		/*
307		* @TODO Create method to use webservice
308		// get result from webservices
309		$result = $this->_get_remote_ppt2lp_files($file);
310		$result = unserialize(base64_decode($result));
311
312		// Save remote images to server
313		chmod($this->base_work_dir.$this->created_dir, api_get_permissions_for_new_directories());
314		if (!empty($result['images'])) {
315		foreach ($result['images'] as $image => $img_data) {
316		$image_path = $this->base_work_dir.$this->created_dir;
317		@file_put_contents($image_path . '/' . $image, base64_decode($img_data));
318		@chmod($image_path . '/' . $image, 0777);
319		}
320		}
321
322		// files info
323		$files = $result['files'];
324		*/
325		}
326
327		if (file_exists($this->base_work_dir . '/' . $this->created_dir)) {
328
329		// Register Files to Document tool
330		$ids[] = add_document(
331		$_course,
332		'/' . $this->created_dir,
333		'file',
334		filesize($this->base_work_dir . '/' . $this->created_dir),
335		$convertedTitle,
336		sprintf(
337		get_lang('FileConvertedFromXToY'),
338		strtoupper($originalPathInfo['extension']),
339		strtoupper($convertedPathInfo['extension'])
340		),
341		0,
342		true,
343		null,
344		api_get_session_id()
345		);
346		chmod($this->base_work_dir, $permissionFolder);
347		}
348
349		return $ids;
350		}
351		}
352

chamilo / chamilo-lms

Push — 1.10.x ( 4a4ede...319c16 )

OpenofficeDocument::_get_remote_ppt2lp_files() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like