Skip to content

Commit 746bbfc

Browse files
authored
fix: CodeClimate 2.0 fixes (#374)
* Codeclimate part 1 * Method complexity threshold 5->7 * Codeclimate style fixes
1 parent eae9926 commit 746bbfc

File tree

6 files changed

+17
-12
lines changed

6 files changed

+17
-12
lines changed

.codeclimate.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
11
version: "2"
2+
checks:
3+
method-complexity:
4+
config:
5+
threshold: 7
26
plugins:
37
rubocop:
48
enabled: true

app/controllers/manage/configs_controller.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,5 +68,4 @@ def get_config
6868
@config.value = HackathonConfig[var]
6969
end
7070
end
71-
7271
end

app/controllers/manage/stats_controller.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,5 +22,4 @@ def mlh_applied_datatable
2222
def mlh_checked_in_datatable
2323
render json: MLHCheckedInDatatable.new(params, view_context: view_context)
2424
end
25-
2625
end

app/controllers/manage/trackable_events_controller.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ def limit_write_access_to_directors
8585
# If the user isn't a director, scope changes only to those they created
8686
def scope_organizer_access
8787
return if current_user.director? || @trackable_event.blank? || @trackable_event.user.blank?
88+
8889
redirect_to manage_trackable_events_path, notice: 'You may not view events you did not create.' if @trackable_event.user != current_user
8990
end
9091
end

app/controllers/questionnaires_controller.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ def create
5959
return redirect_to questionnaires_path, notice: 'Application already exists.'
6060
end
6161
return unless HackathonConfig['accepting_questionnaires']
62+
6263
@questionnaire = Questionnaire.new(convert_school_name_to_id(questionnaire_params))
6364
@questionnaire.user_id = current_user.id
6465

app/models/questionnaire.rb

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -258,16 +258,17 @@ def queue_triggered_email_create
258258
end
259259

260260
def queue_triggered_email_rsvp_reminder
261-
if saved_change_to_acc_status? && acc_status == "accepted"
262-
days_remaining = Date.parse(HackathonConfig["event_start_date"]).in_time_zone.to_date - Time.now.in_time_zone.to_date
263-
if days_remaining > 14
264-
deliver_date = 7.days.from_now
265-
elsif days_remaining > 10
266-
deliver_date = 5.days.from_now
267-
elsif days_remaining > 3
268-
deliver_date = 2.days.from_now
269-
end
270-
UserMailer.rsvp_reminder_email(user_id).deliver_later(wait_until: deliver_date) if deliver_date.present?
261+
return unless saved_change_to_acc_status? && acc_status == "accepted"
262+
263+
event_start = Date.parse(HackathonConfig["event_start_date"]).in_time_zone
264+
days_remaining = event_start.to_date - Time.now.in_time_zone.to_date
265+
if days_remaining > 14
266+
deliver_date = 7.days.from_now
267+
elsif days_remaining > 10
268+
deliver_date = 5.days.from_now
269+
elsif days_remaining > 3
270+
deliver_date = 2.days.from_now
271271
end
272+
UserMailer.rsvp_reminder_email(user_id).deliver_later(wait_until: deliver_date) if deliver_date.present?
272273
end
273274
end

0 commit comments

Comments
 (0)