owncloud/core

File AvatarController.php has 251 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/**
 * @author Joas Schilling <coding@schilljs.com>
 * @author Lukas Reschke <lukas@statuscode.ch>
 * @author Martin Mattel <martin.mattel@diemattels.at>

Function _handleFileTreeEntry has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        _handleFileTreeEntry: function (entry, path) {
            var that = this,
                dfd = $.Deferred(),
                entries = [],
                dirReader,

Function render has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        render: function () {
            this.$el.html(this.template({
                cid: this.cid,
                urlLabel: t('core', 'Link'),
                addLinkText: t('core', 'Create public link'),

Function _getUsersForBatchAction has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        _getUsersForBatchAction: function(search) {
            var view = this;
            var foundUsers = [];
            var notFound = [];
            var promises = [];

Method setEnabled has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function setEnabled($id, $enabled) {
        $userId = $this->userSession->getUser()->getUID();
        $user = $this->userManager->get($id);
        '@phan-var \OC\Group\Manager $this->groupManager';

Method xmlDeserialize has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function xmlDeserialize(Reader $reader) {
        $elems = (array)$reader->parseInnerTree([
            '{DAV:}prop' => KeyValue::class,
            '{http://owncloud.org/ns}filter-rules' => Base::class,
            '{http://owncloud.org/ns}search' => KeyValue::class,

Method createFile has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function createFile($name, $data = null) {
        # the check here is necessary, because createFile uses put covered in sabre/file.php
        # and not touch covered in files/view.php
        if (Filesystem::isForbiddenFileOrDir($name)) {
            throw new SabreForbidden();

Method opendir has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function opendir($path) {
        $folder = $this->getDriveFile($path);
        if ($folder) {
            $files = [];
            $duplicates = [];

Method getVersions has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function getVersions($uid, $filename) {
        $versions = [];
        if ($filename === null || $filename === '') {
            return $versions;
        }

Method parseRetentionObligation has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function parseRetentionObligation() {
        $splitValues = \explode(',', $this->retentionObligation);
        if (!isset($splitValues[0])) {
            $minValue = 'auto';
        } else {

Method getSharedWith has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getSharedWith($userId, $shareType, $node, $limit, $offset) {
        /** @var Share[] $shares */
        $shares = [];

        if ($shareType === \OCP\Share::SHARE_TYPE_USER) {

Method call has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function call($parameters) {
        $request = \OC::$server->getRequest();
        $method = $request->getMethod();

        // Prepare the request variables

Method execute has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function execute(InputInterface $input, OutputInterface $output): int {
        $appSteps = $this->getAppsRepairSteps($output);
        // Handle listing repair steps
        $steps = \array_merge(
            \OC\Repair::getRepairSteps(),

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    tags = _.chain(tags).filter(function(xmlvalue) {
                        return (xmlvalue.namespaceURI === OC.Files.Client.NS_OWNCLOUD && xmlvalue.nodeName.split(':')[1] === 'tag');
                    }).map(function(xmlvalue) {
                        return xmlvalue.textContent || xmlvalue.text;
                    }).value();

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                try {
                    if (!Files.isFileNameValid(filename)) {
                        // Files.isFileNameValid(filename) throws an exception itself
                    } else if (self.fileList.inList(filename)) {
                        throw t('files', '{newname} already exists', {newname: filename});

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                try {
                    checkInput();
                    input.tooltip('hide');
                    input.removeClass('error');
                } catch (error) {

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                try {
                    var newName = input.val().trim();
                    input.tooltip('hide');
                    form.remove();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    data.shareTypes = _.chain(shareTypesProp).filter(function(xmlvalue) {
                        return (xmlvalue.namespaceURI === OC.Files.Client.NS_OWNCLOUD && xmlvalue.nodeName.split(':')[1] === 'share-type');
                    }).map(function(xmlvalue) {
                        return parseInt(xmlvalue.textContent || xmlvalue.text, 10);
                    }).value();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if ($this->offsetGet($path) === null) {
            // TODO: chunking if more than 1000 properties
            $sql = self::SELECT_BY_PATH_STMT;

            $whereValues = [$path];

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if ($this->offsetGet($fileId) === null) {
            // TODO: chunking if more than 1000 properties
            $sql = self::SELECT_BY_ID_STMT;
            $whereValues = [$fileId];
            $whereTypes = [null];

• Read up
  Read up