🚨(back) lint code with pylint 3.3.0

New pylint version 3.3.0 has new rule and code must be fixed.

src/backend/marsha/bbb/utils/bbb_utils.py

@@ -378,7 +378,7 @@ def process_recording(classroom, recording_data):
  )
 
 
-def process_recordings( # pylint: disable=too-many-arguments
+def process_recordings( # pylint: disable=too-many-arguments,too-many-positional-arguments
  classroom,
  recordings_data,
  record_id=None,

src/backend/marsha/core/fields.py

@@ -12,6 +12,7 @@ class InvertedCheckboxSelectMultiple(CheckboxSelectMultiple):
  Modified CheckboxSelectMultiple to invert the choices.
  """
 
+ # pylint: disable=too-many-positional-arguments
  def create_option(
  self, name, value, label, selected, index, subindex=None, attrs=None
  ):

src/backend/marsha/core/lti/validator.py

@@ -90,7 +90,7 @@ def validate_client_key(self, client_key, request):
  oauth_consumer_key=client_key, is_enabled=True
  ).exists()
 
- # pylint: disable=too-many-arguments
+ # pylint: disable=too-many-arguments,too-many-positional-arguments
  def validate_timestamp_and_nonce(
  self,
  client_key: str,

src/backend/marsha/core/management/commands/send_reminders.py

@@ -51,7 +51,7 @@ def query_to_update_step(self, livesession, step):
  )
  )
 
- # pylint: disable=too-many-arguments
+ # pylint: disable=too-many-arguments,too-many-positional-arguments
  def send_reminders_and_update_livesessions_step(
  self, livesessions, step, mail_object, trans_context=None, template="reminder"
  ):

src/backend/marsha/core/simple_jwt/tokens.py

@@ -261,7 +261,7 @@ def for_lti_portability_request(
  return token
 
  @classmethod
- def for_playlist_id( # pylint: disable=too-many-arguments
+ def for_playlist_id( # pylint: disable=too-many-arguments,too-many-positional-arguments
  cls,
  playlist_id,
  session_id,

src/backend/marsha/core/xapi.py

@@ -82,7 +82,7 @@ def build_common_statement_properties(
 class XAPIDocumentStatement(XAPIStatementMixin):
  """Object managing statement for document objects."""
 
- # pylint: disable=too-many-arguments
+ # pylint: disable=too-many-arguments,too-many-positional-arguments
  def _build_statement(self, document, statement, homepage, user=None, user_id=None):
  """Build all common properties for a document."""
 
@@ -182,7 +182,7 @@ def _get_activity_type(self, video):
 
  return activity_type
 
- # pylint: disable=too-many-arguments
+ # pylint: disable=too-many-arguments,too-many-positional-arguments
  def _build_statement(self, video, statement, homepage, user=None, user_id=None):
  """Build all common properties for a video."""
  if re.match(r"^http(s?):\/\/.*", homepage) is None: