owncloud/core

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

Open

    protected function execute(InputInterface $input, OutputInterface $output): int {
        $inputPath = $input->getOption('path');
        $groups = $input->getOption('groups') ? \explode(',', $input->getOption('groups')) : [];
        $groups = \array_unique(\array_merge($groups, $input->getOption('group')));
        $shouldRepairStoragesIndividually = (bool) $input->getOption('repair');

Method fopen has 41 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function fopen($path, $mode) {
        $fullPath = $this->buildPath($path);
        $useExisting = true;
        switch ($mode) {
            case 'r':

Method deleteTags has 41 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function deleteTags($tagIds) {
        if (!\is_array($tagIds)) {
            $tagIds = [$tagIds];
        }

Method getShareById has 41 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getShareById($id, $recipientId = null) {
        if (!ctype_digit($id)) {
            // share id is defined as a field of type integer
            // if someone calls the API asking for a share id like "abc"
            // then there is no point trying to query the database,

Method getSharesBy has 41 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getSharesBy($userId, $shareType, $path = null, $reshares = false, $limit = 50, $offset = 0) {
        if ($path !== null &&
                !($path instanceof \OCP\Files\File) &&
                !($path instanceof \OCP\Files\Folder)) {
            throw new \InvalidArgumentException('invalid path');

Method decryptAllUsersFiles has 41 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function decryptAllUsersFiles($user = '') {
        $this->output->writeln("\n");

        $progress = new ProgressBar($this->output);
        $progress->setFormat(" %message% \n [%bar%]");

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

Open

    protected function execute(InputInterface $input, OutputInterface $output): int {
        $t = new Table($output);
        $displayInvalidJobs = $input->getOption('display-invalid-jobs');
        $headers = ['Job ID', 'Job', 'Job Arguments', 'Last Run', 'Last Checked', 'Reserved At', 'Execution Duration (s)'];
        if ($displayInvalidJobs) {

Method __construct has 13 arguments (exceeds 4 allowed). Consider refactoring.

Open

        $appName,
        IRequest $request,
        IConfig $config,
        IURLGenerator $urlGenerator,
        IUserManager $userManager,

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

Open

                if ($dir === '/' || $dir === '.') {
                    $delimiter = \strrpos($filenameOnly, '.d');
                    $timestamp = \substr($filenameOnly, $delimiter+2);
                    $filenameOnlyWithoutTimestamp = \substr($filenameOnly, 0, $delimiter);
                    $dirAndFilename = $filenameOnlyWithoutTimestamp;

• Read up
  Read up

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

Open

                if ($dir === '/' || $dir === '.') {
                    $delimiter = \strrpos($filenameOnly, '.d');
                    $timestamp = \substr($filenameOnly, $delimiter+2);
                    $filenameOnlyWithoutTimestamp = \substr($filenameOnly, 0, $delimiter);
                    $dirAndFilename = $filenameOnlyWithoutTimestamp;

• Read up
  Read up

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

Open

            case '{'.self::NS_CALENDARSERVER.'}publish-calendar':

                // We can only deal with IShareableCalendar objects
                if (!$node instanceof Calendar) {
                    return;

• Read up
  Read up

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

Open

    public function releaseLock($path, $type, ILockingProvider $provider) {
        /** @var \OCP\Files\Storage $targetStorage */
        list($targetStorage, $targetInternalPath) = $this->resolvePath($path);
        $targetStorage->releaseLock($targetInternalPath, $type, $provider);
        // unlock the parent folders of the owner when unlocking the share as recipient

• Read up
  Read up

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

Open

    public function acquireLock($path, $type, ILockingProvider $provider) {
        /** @var \OCP\Files\Storage $targetStorage */
        list($targetStorage, $targetInternalPath) = $this->resolvePath($path);
        $targetStorage->acquireLock($targetInternalPath, $type, $provider);
        // lock the parent folders of the owner when locking the share as recipient

• Read up
  Read up

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

Open

            case '{'.self::NS_CALENDARSERVER.'}unpublish-calendar':

                // We can only deal with IShareableCalendar objects
                if (!$node instanceof Calendar) {
                    return;

• Read up
  Read up

Function _onClickRestoreSelected has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        _onClickRestoreSelected: function(event) {
            event.preventDefault();
            var self = this;
            var allFiles = this.$el.find('.select-all').is(':checked');
            var files = [];

Function attach has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        attach: function(fileList) {
            this._extendFileActions(fileList);

            var oldCreateRow = fileList._createRow;
            fileList._createRow = function(fileData) {

Function send has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

                send = function () {
                    that._sending += 1;
                    // Set timer for bitrate progress calculation:
                    options._bitrateTimer = new that._BitrateTimer();
                    jqXHR = jqXHR || (

Function _checkSecurityHeaders has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        _checkSecurityHeaders: function(xhr) {
            var messages = [];

            if (xhr.status === 200) {
                var securityHeaders = {

Function _checkSSL has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        _checkSSL: function(xhr) {
            var messages = [];

            if (xhr.status === 200) {
                var docPlaceholderUrl = '';

Function filePath has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    filePath: function (app, type, file) {
        var isCore = OC.coreApps.indexOf(app) !== -1,
            link = OC.getRootPath();
        if (file.substring(file.length - 3) === 'php' && !isCore) {
            link += '/index.php/apps/' + app;