Fix Rubocop offenses

Author: MrSerth

app/controllers/programming_groups_controller.rb

@@ -38,16 +38,17 @@ def edit
   end
 
   def create
-    programming_partner_ids = programming_group_params&.fetch(:programming_partner_ids, [])&.split(',')&.map(&:strip)&.uniq
-    users = programming_partner_ids&.map do |partner_id|
+    programming_partner_ids = []
+    programming_partner_ids = programming_group_params.fetch(:programming_partner_ids, '').split(',').map(&:strip).uniq if programming_group_params
+    users = programming_partner_ids.map do |partner_id|
       User.find_by_id_with_type(partner_id)
     rescue ActiveRecord::RecordNotFound
       partner_id
     end
     @programming_group = ProgrammingGroup.new(exercise: @exercise, users:)
     authorize!
 
-    unless programming_partner_ids&.include? current_user.id_with_type
+    unless programming_partner_ids.include? current_user.id_with_type
       @programming_group.add(current_user)
     end
 

app/controllers/submissions_controller.rb

@@ -268,8 +268,8 @@ def score
     # send_hints(client_socket, StructuredError.where(submission: @submission))
 
     # Finally, send the score to the LTI consumer and check for notifications
-    check_submission(send_scores(@submission)).compact.each do |notification|
-      client_socket&.send_data(notification&.merge(cmd: :status)&.to_json)
+    check_submission(send_scores(@submission)).compact_blank.each do |notification|
+      client_socket&.send_data(notification.merge(cmd: :status).to_json)
     end
   rescue Runner::Error::RunnerInUse => e
     extract_durations(e)

app/models/code_ocean/file.rb

@@ -180,9 +180,9 @@ def missing_read_permissions?
 
       # We use a bitwise AND with the permission bits and compare that to zero
       if privileged_execution.present?
-        (permissions & OWNER_READ_PERMISSION).zero?
+        permissions.nobits?(OWNER_READ_PERMISSION)
       else
-        (permissions & OTHER_READ_PERMISSION).zero?
+        permissions.nobits?(OTHER_READ_PERMISSION)
       end
     end
   end

lib/file_tree.rb

@@ -111,7 +111,9 @@ def name(node)
   private :name
 
   def path(node)
-    "#{node.parentage&.reverse&.drop(1)&.map(&:name)&.join('/')}/#{node.name}"
+    path = node.parentage
+    path = "/#{path.reverse.drop(1).map(&:name).join('/')}" unless path.nil?
+    "#{path}/#{node.name}"
   end
   private :path