From fc80b28bbce398b29b56a2d365af56cdd79eb531 Mon Sep 17 00:00:00 2001 From: Stu Young Date: Wed, 15 May 2019 15:01:53 -0400 Subject: [PATCH] INCR-304 Run python-modernize on lms/djangoapps/discussion/management (#20622) * run python modernize * run isort --- lms/djangoapps/discussion/management/commands/assign_role.py | 3 ++- .../management/commands/assign_roles_for_course.py | 3 ++- .../management/commands/create_roles_for_existing.py | 3 ++- .../discussion/management/commands/get_discussion_link.py | 5 ++++- .../discussion/management/commands/reload_forum_users.py | 3 ++- .../discussion/management/commands/seed_permissions_roles.py | 3 +++ .../discussion/management/commands/show_permissions.py | 2 +- .../discussion/management/commands/sync_user_info.py | 3 +++ 8 files changed, 19 insertions(+), 6 deletions(-) diff --git a/lms/djangoapps/discussion/management/commands/assign_role.py b/lms/djangoapps/discussion/management/commands/assign_role.py index 2c42d8755a..ac8858604b 100644 --- a/lms/djangoapps/discussion/management/commands/assign_role.py +++ b/lms/djangoapps/discussion/management/commands/assign_role.py @@ -1,8 +1,9 @@ # pylint: disable=missing-docstring -from __future__ import print_function +from __future__ import absolute_import, print_function from django.contrib.auth.models import User from django.core.management.base import BaseCommand + from openedx.core.djangoapps.django_comment_common.models import Role diff --git a/lms/djangoapps/discussion/management/commands/assign_roles_for_course.py b/lms/djangoapps/discussion/management/commands/assign_roles_for_course.py index 4f54de64ca..1da283f4c5 100644 --- a/lms/djangoapps/discussion/management/commands/assign_roles_for_course.py +++ b/lms/djangoapps/discussion/management/commands/assign_roles_for_course.py @@ -5,9 +5,10 @@ This must be run only after seed_permissions_roles.py! Creates default roles for all users in the provided course. Just runs through Enrollments. """ -from __future__ import print_function +from __future__ import absolute_import, print_function from django.core.management.base import BaseCommand + from openedx.core.djangoapps.django_comment_common.models import assign_default_role_on_enrollment from student.models import CourseEnrollment diff --git a/lms/djangoapps/discussion/management/commands/create_roles_for_existing.py b/lms/djangoapps/discussion/management/commands/create_roles_for_existing.py index 04d32da224..307f3f0966 100644 --- a/lms/djangoapps/discussion/management/commands/create_roles_for_existing.py +++ b/lms/djangoapps/discussion/management/commands/create_roles_for_existing.py @@ -4,9 +4,10 @@ This must be run only after seed_permissions_roles.py! Creates default roles for all users currently in the database. Just runs through Enrollments. """ -from __future__ import print_function +from __future__ import absolute_import, print_function from django.core.management.base import BaseCommand + from openedx.core.djangoapps.django_comment_common.models import assign_default_role_on_enrollment from student.models import CourseEnrollment diff --git a/lms/djangoapps/discussion/management/commands/get_discussion_link.py b/lms/djangoapps/discussion/management/commands/get_discussion_link.py index 455ada839a..f275ecdfb3 100644 --- a/lms/djangoapps/discussion/management/commands/get_discussion_link.py +++ b/lms/djangoapps/discussion/management/commands/get_discussion_link.py @@ -1,8 +1,11 @@ # pylint: disable=missing-docstring -from courseware.courses import get_course +from __future__ import absolute_import + from django.core.management.base import BaseCommand, CommandError from opaque_keys.edx.keys import CourseKey +from courseware.courses import get_course + class Command(BaseCommand): help = 'Write a discussion link for a given course on standard output.' diff --git a/lms/djangoapps/discussion/management/commands/reload_forum_users.py b/lms/djangoapps/discussion/management/commands/reload_forum_users.py index 402891ddaa..2d5868193c 100644 --- a/lms/djangoapps/discussion/management/commands/reload_forum_users.py +++ b/lms/djangoapps/discussion/management/commands/reload_forum_users.py @@ -2,10 +2,11 @@ """ Reload forum (comment client) users from existing users. """ -from __future__ import print_function +from __future__ import absolute_import, print_function from django.contrib.auth.models import User from django.core.management.base import BaseCommand + import openedx.core.djangoapps.django_comment_common.comment_client as cc diff --git a/lms/djangoapps/discussion/management/commands/seed_permissions_roles.py b/lms/djangoapps/discussion/management/commands/seed_permissions_roles.py index 6d5529d5ec..7bc0d0f9b2 100644 --- a/lms/djangoapps/discussion/management/commands/seed_permissions_roles.py +++ b/lms/djangoapps/discussion/management/commands/seed_permissions_roles.py @@ -2,8 +2,11 @@ """ Management command to seed default permissions and roles. """ +from __future__ import absolute_import + from django.core.management.base import BaseCommand from opaque_keys.edx.keys import CourseKey + from openedx.core.djangoapps.django_comment_common.utils import seed_permissions_roles diff --git a/lms/djangoapps/discussion/management/commands/show_permissions.py b/lms/djangoapps/discussion/management/commands/show_permissions.py index 609e6a264f..fcd52ca662 100644 --- a/lms/djangoapps/discussion/management/commands/show_permissions.py +++ b/lms/djangoapps/discussion/management/commands/show_permissions.py @@ -1,5 +1,5 @@ # pylint: disable=missing-docstring,too-many-format-args -from __future__ import print_function +from __future__ import absolute_import, print_function from django.contrib.auth.models import User from django.core.management.base import BaseCommand diff --git a/lms/djangoapps/discussion/management/commands/sync_user_info.py b/lms/djangoapps/discussion/management/commands/sync_user_info.py index 43a553f355..2de2f167ba 100644 --- a/lms/djangoapps/discussion/management/commands/sync_user_info.py +++ b/lms/djangoapps/discussion/management/commands/sync_user_info.py @@ -3,8 +3,11 @@ One-off script to sync all user information to the discussion service (later info will be synced automatically) """ +from __future__ import absolute_import + from django.contrib.auth.models import User from django.core.management.base import BaseCommand + import openedx.core.djangoapps.django_comment_common.comment_client as cc