Sign upLogin

xybu/onedrived-dev

View on GitHub
Star

Showing 166 of 166 total issues

File merge_dir.py has 467 lines of code (exceeds 250 allowed). Consider refactoring.
Open

import itertools
import logging
import os
import shutil
 
 
Severity: Minor
Found in onedrived/od_tasks/merge_dir.py - About 7 hrs to fix

    Function _handle_remote_file_with_record has a Cognitive Complexity of 42 (exceeds 5 allowed). Consider refactoring.
    Open

        def _handle_remote_file_with_record(self, remote_item, item_record, item_stat, item_local_abspath, all_local_items):
            """
            :param onedrivesdk.model.item.Item remote_item:
            :param onedrived.od_repo.ItemRecord item_record:
            :param posix.stat_result | None item_stat:
    Severity: Minor
    Found in onedrived/od_tasks/merge_dir.py - About 6 hrs to fix

    File od_pref.py has 408 lines of code (exceeds 250 allowed). Consider refactoring.
    Open

    #!/usr/bin/env python3
     
    import json
    import locale
    import os
    Severity: Minor
    Found in onedrived/od_pref.py - About 5 hrs to fix

      Cyclomatic complexity is too high in method _handle_remote_file_with_record. (27)
      Open

          def _handle_remote_file_with_record(self, remote_item, item_record, item_stat, item_local_abspath, all_local_items):
              """
              :param onedrivesdk.model.item.Item remote_item:
              :param onedrived.od_repo.ItemRecord item_record:
              :param posix.stat_result | None item_stat:
      Severity: Minor
      Found in onedrived/od_tasks/merge_dir.py by radon

      Function set_drive has a Cognitive Complexity of 34 (exceeds 5 allowed). Consider refactoring.
      Open

      def set_drive(drive_id=None, email=None, local_root=None, ignore_file=None):
          try:
              all_drives, drive_table = print_all_drives()
              click.echo()
          except Exception as e:
      Severity: Minor
      Found in onedrived/od_pref.py - About 5 hrs to fix

      File od_watcher.py has 365 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      import logging
      import os
      import threading
       
      import onedrivesdk.error
      Severity: Minor
      Found in onedrived/od_watcher.py - About 4 hrs to fix

        Cyclomatic complexity is too high in function set_drive. (23)
        Open

        @click.command(name='set', short_help='Add a remote Drive to sync with local directory or modify an existing one. '
                                              'If either --drive-id or --email is missing, use interactive mode.')
        @click.option('--drive-id', '-d', type=str, required=False, default=None,
                      help='ID of the Drive.')
        @click.option('--email', '-e', type=str, required=False, default=None,
        Severity: Minor
        Found in onedrived/od_pref.py by radon

        Cyclomatic complexity is too high in method handle_event. (22)
        Open

            def handle_event(self, ev, flags, move_pairs):
                """
                :param inotify_simple.Event ev:
                :param [inotify_simple.flags] flags:
                :param dict[int, [inotify_simple.Event, inotify_simple.flags]] move_pairs:
        Severity: Minor
        Found in onedrived/od_watcher.py by radon

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

                    if self.assume_remote_unchanged:
                        logging.info('Remote item for local file "%s" is a directory that has been deleted locally. '
                                     'Delete the remote item and upload the file.', item_local_abspath)
                        if not delete_item.DeleteRemoteItemTask(
                                repo=self.repo, task_pool=self.task_pool, parent_relpath=self.rel_path,
        Severity: Major
        Found in onedrived/od_tasks/merge_dir.py and 1 other location - About 4 hrs to fix
        onedrived/od_tasks/merge_dir.py on lines 451..461

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

                elif item_record is not None:
                    if self.assume_remote_unchanged:
                        logging.info('Remote item for local dir "%s" is a file that has been deleted locally. '
                                     'Delete the remote item and upload the file.', item_local_abspath)
                        if not delete_item.DeleteRemoteItemTask(
        Severity: Major
        Found in onedrived/od_tasks/merge_dir.py and 1 other location - About 4 hrs to fix
        onedrived/od_tasks/merge_dir.py on lines 508..517

        Cyclomatic complexity is too high in method _handle_unpaired_move_to. (22)
        Open

            def _handle_unpaired_move_to(self, to_ev, to_flags, to_repo,
                                         to_parent_dir=None, to_parent_relpath=None):
         
                if to_parent_dir is None:
                    to_parent_dir = self.watch_descriptors[to_ev.wd]
        Severity: Minor
        Found in onedrived/od_watcher.py by radon

        Function _handle_local_file has a Cognitive Complexity of 26 (exceeds 5 allowed). Consider refactoring.
        Open

            def _handle_local_file(self, item_name, item_record, item_stat, item_local_abspath):
                """
                :param str item_name:
                :param onedrived.od_repo.ItemRecord | None item_record:
                :param posix.stat_result | None item_stat:
        Severity: Minor
        Found in onedrived/od_tasks/merge_dir.py - About 3 hrs to fix

        Function _handle_unpaired_move_to has a Cognitive Complexity of 26 (exceeds 5 allowed). Consider refactoring.
        Open

            def _handle_unpaired_move_to(self, to_ev, to_flags, to_repo,
                                         to_parent_dir=None, to_parent_relpath=None):
         
                if to_parent_dir is None:
                    to_parent_dir = self.watch_descriptors[to_ev.wd]
        Severity: Minor
        Found in onedrived/od_watcher.py - About 3 hrs to fix

        Cyclomatic complexity is too high in method _handle_move_pair. (16)
        Open

            def _handle_move_pair(self, move_pair):
                """
                :param [[inotify_simple.Event, inotify_simple.flags], [inotify_simple.Event, inotify_simple.flags]] move_pair:
                """
                from_tup, to_tup = move_pair
        Severity: Minor
        Found in onedrived/od_watcher.py by radon

        Cyclomatic complexity is too high in method _handle_local_file. (16)
        Open

            def _handle_local_file(self, item_name, item_record, item_stat, item_local_abspath):
                """
                :param str item_name:
                :param onedrived.od_repo.ItemRecord | None item_record:
                :param posix.stat_result | None item_stat:
        Severity: Minor
        Found in onedrived/od_tasks/merge_dir.py by radon

        Function _handle_move_pair has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.
        Open

            def _handle_move_pair(self, move_pair):
                """
                :param [[inotify_simple.Event, inotify_simple.flags], [inotify_simple.Event, inotify_simple.flags]] move_pair:
                """
                from_tup, to_tup = move_pair
        Severity: Minor
        Found in onedrived/od_watcher.py - About 3 hrs to fix

        Function handle_event has a Cognitive Complexity of 22 (exceeds 5 allowed). Consider refactoring.
        Open

            def handle_event(self, ev, flags, move_pairs):
                """
                :param inotify_simple.Event ev:
                :param [inotify_simple.flags] flags:
                :param dict[int, [inotify_simple.Event, inotify_simple.flags]] move_pairs:
        Severity: Minor
        Found in onedrived/od_watcher.py - About 3 hrs to fix

        Cyclomatic complexity is too high in function delete_account. (13)
        Open

        @click.command(name='del', short_help='De-authorize and delete an existing account from onedrived.')
        @click.option('--yes', '-y', is_flag=True, default=False, required=False,
                      help='If set, do not ask for confirmation but simply delete if account exists.')
        @click.option('--index', '-i', type=int, required=False, default=None,
                      help='Specify the account to delete by row index in account list table.')
        Severity: Minor
        Found in onedrived/od_pref.py by radon

        Cyclomatic complexity is too high in method handle. (12)
        Open

            def handle(self):
                if not os.path.isdir(self.local_abspath):
                    logging.error('Error: Local path "%s" is not a directory.' % self.local_abspath)
                    return
         
         
        Severity: Minor
        Found in onedrived/od_tasks/merge_dir.py by radon

        Cyclomatic complexity is too high in function set_config. (11)
        Open

        @click.command(name='set', short_help=translator['od_pref.set_config.short_help'])
        @click.argument('key', type=click.Choice(sorted(config_schema.keys())))
        @click.argument('value')
        def set_config(key, value):
            try:
        Severity: Minor
        Found in onedrived/od_pref.py by radon
        Severity
        Category
        Status
        Source
        Language