fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

fetchMedia accesses the super-global variable $_FILES.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

The class XoopsMediaUploader has an overall complexity of 76 which is very high. The configured complexity threshold is 50.

Open

class XoopsMediaUploader {
    /**
    * Flag indicating if unrecognized mimetypes should be allowed (use with precaution ! may lead to security issues )
    **/
    var $allowUnknownTypes = false;

• Exclude checks
  • Disable engine
  • Disable check

File uploader.php has 274 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
// $Id: uploader.php 507 2006-05-26 23:39:35Z skalpa $
//  ------------------------------------------------------------------------ //
//                XOOPS - PHP Content Management System                      //
//                    Copyright (c) 2000 XOOPS.org                           //

XoopsMediaUploader has 21 functions (exceeds 20 allowed). Consider refactoring.

Open

class XoopsMediaUploader {
    /**
    * Flag indicating if unrecognized mimetypes should be allowed (use with precaution ! may lead to security issues )
    **/
    var $allowUnknownTypes = false;

Method fetchMedia has 47 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

The class XoopsMediaUploader has 21 fields. Consider redesigning XoopsMediaUploader to keep the number of fields under 15.

Open

class XoopsMediaUploader {
    /**
    * Flag indicating if unrecognized mimetypes should be allowed (use with precaution ! may lead to security issues )
    **/
    var $allowUnknownTypes = false;

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

class Person {
   protected $one;
   private $two;
   private $three;
   [... many more fields ...]
}

Method upload has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function upload($chmod = 0644) {
        if ($this->uploadDir == '') {
            $this->setErrors( _ER_UP_UPLOADDIRNOTSET );
            return false;
        }

Function fetchMedia has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

• Read up
  Read up

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

Open

    function upload($chmod = 0644) {
        if ($this->uploadDir == '') {
            $this->setErrors( _ER_UP_UPLOADDIRNOTSET );
            return false;
        }

• Read up
  Read up

Method _copyFile has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _copyFile($chmod) {
        $matched = array();
        if (!preg_match("/\.([a-zA-Z0-9]+)$/", $this->mediaName, $matched)) {
            return false;
        }

Function _copyFile has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    function _copyFile($chmod) {
        $matched = array();
        if (!preg_match("/\.([a-zA-Z0-9]+)$/", $this->mediaName, $matched)) {
            return false;
        }

• Read up
  Read up

Method XoopsMediaUploader has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

Function getErrors has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    function &getErrors($ashtml = true) {
        if (!$ashtml) {
            return $this->errors;
        } else {
            $ret = '';

• Read up
  Read up

Avoid too many return statements within this method.

Open

    return true;

Avoid too many return statements within this method.

Open

        return true;

Avoid too many return statements within this method.

Open

        return false;

Avoid too many return statements within this method.

Open

        return false;

Function checkImageType has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    function checkImageType() {
        if(empty($this->checkImageType)) return true;

        if( ("image" == substr($this->mediaType, 0, strpos($this->mediaType, "/"))) ||
            (!empty($this->mediaRealType) && "image" == substr($this->mediaRealType, 0, strpos($this->mediaRealType, "/"))) ){

• Read up
  Read up

The method fetchMedia() has an NPath complexity of 960. The configured NPath complexity threshold is 200.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method upload() has an NPath complexity of 1024. The configured NPath complexity threshold is 200.

Open

    function upload($chmod = 0644) {
        if ($this->uploadDir == '') {
            $this->setErrors( _ER_UP_UPLOADDIRNOTSET );
            return false;
        }

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

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method fetchMedia() has a Cyclomatic Complexity of 15. The configured cyclomatic complexity threshold is 10.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method upload() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.

Open

    function upload($chmod = 0644) {
        if ($this->uploadDir == '') {
            $this->setErrors( _ER_UP_UPLOADDIRNOTSET );
            return false;
        }

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

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method getErrors has a boolean flag argument $ashtml, which is a certain sign of a Single Responsibility Principle violation.

Open

    function &getErrors($ashtml = true) {

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

class Foo {
    public function bar($flag = true) {
    }
}

Remove error control operator '@' on line 373.

Open

    function _copyFile($chmod) {
        $matched = array();
        if (!preg_match("/\.([a-zA-Z0-9]+)$/", $this->mediaName, $matched)) {
            return false;
        }

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

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

Remove error control operator '@' on line 467.

Open

    function checkImageType() {
        if(empty($this->checkImageType)) return true;

        if( ("image" == substr($this->mediaType, 0, strpos($this->mediaType, "/"))) ||
            (!empty($this->mediaRealType) && "image" == substr($this->mediaRealType, 0, strpos($this->mediaRealType, "/"))) ){

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

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

Remove error control operator '@' on line 376.

Open

    function _copyFile($chmod) {
        $matched = array();
        if (!preg_match("/\.([a-zA-Z0-9]+)$/", $this->mediaName, $matched)) {
            return false;
        }

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

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

Remove error control operator '@' on line 381.

Open

    function _copyFile($chmod) {
        $matched = array();
        if (!preg_match("/\.([a-zA-Z0-9]+)$/", $this->mediaName, $matched)) {
            return false;
        }

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

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

The method fetchMedia uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $media_name =& $_FILES[$media_name];
        $this->mediaName = (get_magic_quotes_gpc()) ? stripslashes($media_name['name']) : $media_name['name'];
        $this->mediaName = $media_name['name'];
        $this->mediaType = $media_name['type'];

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '467', column '13').

Open

    function checkImageType() {
        if(empty($this->checkImageType)) return true;

        if( ("image" == substr($this->mediaType, 0, strpos($this->mediaType, "/"))) ||
            (!empty($this->mediaRealType) && "image" == substr($this->mediaRealType, 0, strpos($this->mediaRealType, "/"))) ){

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

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '406', column '17').

Open

    function checkMaxWidth() {
        if (!isset($this->maxWidth)) {
            return true;
        }
        if (false !== $dimension = getimagesize($this->mediaTmpName)) {

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

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method _copyFile uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $this->savedFileName = strtolower($this->mediaName);
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method checkMaxWidth uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            trigger_error(sprintf('Failed fetching image size of %s, skipping max width check..', $this->mediaTmpName), E_USER_WARNING);
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '425', column '17').

Open

    function checkMaxHeight() {
        if (!isset($this->maxHeight)) {
            return true;
        }
        if (false !== $dimension = getimagesize($this->mediaTmpName)) {

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

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method getErrors uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $ret = '';
            if (count($this->errors) > 0) {
                $ret = '<h4>Errors Returned While Uploading</h4>';
                foreach ($this->errors as $error) {

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid unused local variables such as '$info'.

Open

            if ( ! ( $info = @getimagesize( $this->mediaTmpName ) ) ) {

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

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

Avoid excessively long variable names like $extensionsToBeSanitized. Keep variable name length under 20.

Open

    var $extensionsToBeSanitized = array( 'php' , 'phtml' , 'phtm' , 'php3' , 'php4' , 'cgi' , 'pl' , 'asp', 'php5' );

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

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class XoopsMediaUploader {

• Exclude checks
  • Disable engine
  • Disable check

The parameter $media_name is not named in camelCase.

Open

    function fetchMedia($media_name, $index = null) {
/*
        if ( empty( $this->extensionToMime ) ) {
*/
        if ( count( $this->extensionToMime ) == 0 ) {

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

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

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

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $maxHeight;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $savedDestination;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$allowUnknownTypes"

Open

    var $allowUnknownTypes = false;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$mediaTmpName"

Open

    var $mediaTmpName;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $mediaRealType = '';

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $targetFileName;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$prefix"

Open

    var $prefix;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$mediaRealType"

Open

    var $mediaRealType = '';

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$savedFileName"

Open

    var $savedFileName;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $uploadDir = '';

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$errors"

Open

    var $errors = array();

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $extensionToMime = array();

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $allowedMimeTypes = array();

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $errors = array();

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $allowUnknownTypes = false;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$mediaError"

Open

    var $mediaError;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$checkImageType"

Open

    var $checkImageType = true;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $mediaType;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$mediaType"

Open

    var $mediaType;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$mediaSize"

Open

    var $mediaSize;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $mediaTmpName;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $mediaError;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$maxHeight"

Open

    var $maxHeight;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $prefix;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "fetchMedia"

Open

    function fetchMedia($media_name, $index = null) {

• Exclude checks
  • Disable engine
  • Disable check

Opening brace of a class must be on the line after the definition

Open

class XoopsMediaUploader {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$extensionsToBeSanitized"

Open

    var $extensionsToBeSanitized = array( 'php' , 'phtml' , 'phtm' , 'php3' , 'php4' , 'cgi' , 'pl' , 'asp', 'php5' );

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $imageExtensions = array( 1 => 'gif', 2 => 'jpg', 3 => 'png', 4 => 'swf', 5 => 'psd', 6 => 'bmp', 7 => 'tif', 8 => 'tif', 9 => 'jpc', 10 => 'jp2', 11 => 'jpx', 12 => 'jb2', 13 => 'swc', 14 => 'iff', 15 => 'wbmp', 16 => 'xbm' );

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "XoopsMediaUploader"

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( count( $this->extensionToMime ) == 0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "getMediaName"

Open

    function getMediaName() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

if ( file_exists(XOOPS_ROOT_PATH . '/language/' . $GLOBALS['xoopsConfig']['language'] . '/uploader_error.php') ) {

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $mediaName;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$uploadDir"

Open

    var $uploadDir = '';

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$extensionToMime"

Open

    var $extensionToMime = array();

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$mediaName"

Open

    var $mediaName;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$savedDestination"

Open

    var $savedDestination;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $savedFileName;

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between default value and equals sign for argument "$maxWidth"; expected 1 but found 0

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between argument "$maxHeight" and equals sign; expected 1 but found 0

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $mediaSize;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$imageExtensions"

Open

    var $imageExtensions = array( 1 => 'gif', 2 => 'jpg', 3 => 'png', 4 => 'swf', 5 => 'psd', 6 => 'bmp', 7 => 'tif', 8 => 'tif', 9 => 'jpc', 10 => 'jp2', 11 => 'jpx', 12 => 'jb2', 13 => 'swc', 14 => 'iff', 15 => 'wbmp', 16 => 'xbm' );

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between argument "$maxWidth" and equals sign; expected 1 but found 0

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$allowedMimeTypes"

Open

    var $allowedMimeTypes = array();

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $maxFileSize = 0;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$maxFileSize"

Open

    var $maxFileSize = 0;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $maxWidth;

• Exclude checks
  • Disable engine
  • Disable check

Method name "XoopsMediaUploader::XoopsMediaUploader" is not in camel caps format

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "getMediaType"

Open

    function getMediaType() {

• Exclude checks
  • Disable engine
  • Disable check

Space found before comma in function call

Open

        $ext = strtolower( substr( strrchr( $this->savedDestination , '.' ) , 1 ) ) ;

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between argument "$maxFileSize" and equals sign; expected 1 but found 0

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "getSavedDestination"

Open

    function getSavedDestination(){

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$targetFileName"

Open

    var $targetFileName;

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between default value and equals sign for argument "$maxHeight"; expected 1 but found 0

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on property "$maxWidth"

Open

    var $maxWidth;

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between default value and equals sign for argument "$maxFileSize"; expected 1 but found 0

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "getMediaSize"

Open

    function getMediaSize() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "getMediaTmpName"

Open

    function getMediaTmpName() {

• Exclude checks
  • Disable engine
  • Disable check

Space found before comma in function call

Open

        $ext = strtolower( substr( strrchr( $this->savedDestination , '.' ) , 1 ) ) ;

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $extensionsToBeSanitized = array( 'php' , 'phtml' , 'phtm' , 'php3' , 'php4' , 'cgi' , 'pl' , 'asp', 'php5' );

• Exclude checks
  • Disable engine
  • Disable check

The var keyword must not be used to declare a property

Open

    var $checkImageType = true;

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

    if ( array_key_exists($this->mediaExt, $this->extensionToMime) ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "setTargetFileName"

Open

    function setTargetFileName($value) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "setPrefix"

Open

    function setPrefix($value){

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "getSavedFileName"

Open

    function getSavedFileName(){

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "upload"

Open

    function upload($chmod = 0644) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "checkMaxHeight"

Open

    function checkMaxHeight() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "checkImageType"

Open

    function checkImageType() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "checkMaxWidth"

Open

    function checkMaxWidth() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( count( $this->extensionToMime ) == 0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "_copyFile"

Open

    function _copyFile($chmod) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before comma in function call

Open

        if( in_array( $ext , $this->imageExtensions ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( ! ( $info = @getimagesize( $this->mediaTmpName ) ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at start of control structure

Open

            (!empty($this->mediaRealType) && "image" == substr($this->mediaRealType, 0, strpos($this->mediaRealType, "/"))) ){

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "GetextensionToMime"

Open

    function GetextensionToMime() {

• Exclude checks
  • Disable engine
  • Disable check

Space found before comma in function call

Open

        $xoopsDB->prefix("mimetypes") . ' t on p.mperm_mime = t.mime_id' . ' WHERE p.mperm_module=' . $xoopsModule->mid() . ' AND p.mperm_groups IN (' . implode(',' , $groups) . ')' . ' GROUP BY t.mime_ext' ;

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "sanitizeMultipleExtensions"

Open

    function sanitizeMultipleExtensions() {

• Exclude checks
  • Disable engine
  • Disable check

Method name "XoopsMediaUploader::GetextensionToMime" is not in camel caps format

Open

    function GetextensionToMime() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if( in_array( $ext , $this->imageExtensions ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        while ( $myrow = $xoopsDB->fetchArray($result) ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "setErrors"

Open

    function setErrors($error) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "checkMimeType"

Open

    function checkMimeType() {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    function _copyFile($chmod) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if (!is_object ( $xoopsModule ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if( $info === false || $this->imageExtensions[ (int)$info[2] ] != $ext ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "checkMaxFileSize"

Open

    function checkMaxFileSize() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "getErrors"

Open

    function &getErrors($ashtml = true) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $mediaError;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    include_once XOOPS_ROOT_PATH . '/language/' . $GLOBALS['xoopsConfig']['language'] . '/uploader_error.php';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $mediaType;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $mediaTmpName;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Flag indicating if unrecognized mimetypes should be allowed (use with precaution ! may lead to security issues )

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $maxHeight;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->allowedMimeTypes =& $allowedMimeTypes;

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 144 characters

Open

        $this->mediaName = (get_magic_quotes_gpc()) ? stripslashes($_FILES[$media_name]['name'][$index]) : $_FILES[$media_name]['name'][$index];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaTmpName = $_FILES[$media_name]['tmp_name'][$index];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaTmpName = $media_name['tmp_name'];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $mediaName;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $uploadDir = '';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   array   $allowedMimeTypes

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->setErrors( _ER_UP_FILENOTFOUND );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->setErrors( _ER_UP_FILENAMEEMPTY );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function getSavedDestination(){

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if ($this->uploadDir == '') {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!is_dir($this->uploadDir)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $allowedMimeTypes = array();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $savedFileName;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $checkImageType = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @return  bool

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (count($this->errors) > 0) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $allowUnknownTypes = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   int$maxHeight

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( sprintf(_ER_UP_FILEHEIGHTTOOLARGE, $this->maxHeight) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $maxFileSize = 0;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->maxFileSize = intval($maxFileSize);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function fetchMedia($media_name, $index = null) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( _ER_UP_MIMETYPELOAD );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaType = $media_name['type'];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    if ($this->mediaTmpName == 'none' || !is_uploaded_file($this->mediaTmpName)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    if ($this->mediaError > 0) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   string  $value

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Get the type of the uploaded file

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return $this->mediaSize;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!$this->checkMaxHeight()) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $imageExtensions = array( 1 => 'gif', 2 => 'jpg', 3 => 'png', 4 => 'swf', 5 => 'psd', 6 => 'bmp', 7 => 'tif', 8 => 'tif', 9 => 'jpc', 10 => 'jp2', 11 => 'jpx', 12 => 'jb2', 13 => 'swc', 14 => 'iff', 15 => 'wbmp', 16 => 'xbm' );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (is_array($allowedMimeTypes)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    if (!isset($_FILES[$media_name])) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    } elseif (is_array($_FILES[$media_name]['name']) && isset($index)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->setErrors( _ER_UP_NOFILEUPLOADED );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return $this->mediaName;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Get the size of the uploaded file

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @return  int

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function getMediaTmpName() {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( _ER_UP_UPLOADDIRNOTSET );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->sanitizeMultipleExtensions();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if ( count( $this->extensionToMime ) == 0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaError = !empty($media_name['error']) ? $media_name['error'] : 0;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @return  string

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return $this->savedFileName;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( sprintf(_ER_UP_FILEWIDTHTOOLARGE, $this->maxWidth) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $mediaSize;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $savedDestination;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $index = intval($index);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    $this->errors = array();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @return  string

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @return  string

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!is_writeable($this->uploadDir)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( sprintf(_ER_UP_FAILEDUPLOADFILE, $this->mediaName) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $targetFileName;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $prefix;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $extensionsToBeSanitized = array( 'php' , 'phtml' , 'phtm' , 'php3' , 'php4' , 'cgi' , 'pl' , 'asp', 'php5' );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   int$maxFileSize

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $media_name =& $_FILES[$media_name];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    return true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   string  $value

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @return  bool

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function upload($chmod = 0644) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( sprintf(_ER_UP_FAILEDOPENDIR, $this->uploadDir) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!$this->checkMaxFileSize()) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->maxWidth = intval($maxWidth);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->GetextensionToMime();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaError = !empty($_FILES[$media_name]['error'][$index]) ? $_FILES[$media_name]['error'][$index] : 0;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Set the target filename

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function setTargetFileName($value) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Check the file and copy it to the destination

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!$this->_copyFile($chmod)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $extensionToMime = array();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   string  $uploadDir

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize=0, $maxWidth=null, $maxHeight=null) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaType = $_FILES[$media_name]['type'][$index];

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 137 characters

Open

            //trigger_error( "XoopsMediaUploader: Set mediaRealType to {$this->mediaRealType} (file extension is $ext)", E_USER_NOTICE );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->setErrors(  _ER_UP_INVALIDFILESIZE );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->targetFileName = strval(trim($value));

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function getSavedFileName(){

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!$this->checkImageType()) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    include_once XOOPS_ROOT_PATH . '/language/english/uploader_error.php';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $mediaRealType = '';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $errors = array();

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 235 characters

Open

    var $imageExtensions = array( 1 => 'gif', 2 => 'jpg', 3 => 'png', 4 => 'swf', 5 => 'psd', 6 => 'bmp', 7 => 'tif', 8 => 'tif', 9 => 'jpc', 10 => 'jp2', 11 => 'jpx', 12 => 'jb2', 13 => 'swc', 14 => 'iff', 15 => 'wbmp', 16 => 'xbm' );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Constructor

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

 * $uploader = new XoopsMediaUploader('/home/xoops/uploads', $allowed_mimetypes, $maxfilesize, $maxfilewidth, $maxfileheight);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Fetch the uploaded file

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    var $maxWidth;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    $this->mediaExt = strtolower(substr( $this->mediaName, strrpos( $this->mediaName, '.' ) + 1 ));

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    if ( array_key_exists($this->mediaExt, $this->extensionToMime) ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaSize = $media_name['size'];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->maxFileSize = $this->extensionToMime[$this->mediaExt]['maxSize'];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( sprintf(_ER_UP_FILESIZETOOLARGE, $this->mediaSize) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->setErrors( sprintf(_ER_UP_ERROROCCURRED, $this->mediaError) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * Set the prefix

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @return  string

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!$this->checkMimeType()) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( sprintf(_ER_UP_MIMETYPENOTALLOWED, $this->mediaType) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    // extensions needed image check (anti-IE Content-Type XSS)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   int$maxWidth

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   int$cmodvalue

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    * @param   string  $media_name Name of the file field

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaName = (get_magic_quotes_gpc()) ? stripslashes($media_name['name']) : $media_name['name'];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->mediaName = $media_name['name'];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->maxWidth = $this->extensionToMime[$this->mediaExt]['maxWidth'];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    **/

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->setErrors( sprintf(_ER_UP_FAILEDOPENDIRWRITE, $this->uploadDir) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if (!$this->checkMaxWidth()) {

• Exclude checks
  • Disable engine
  • Disable check