814b7cee-7846-4f1b-9594-6ee9d5c461bc

Committed 30 Jun 2025 09:33PM UTC coverage: 97.424% (+0.01%) from 97.411%

Build # 814b7cee-7846-4f1b-9594-6ee9d5c461bc

Build Type

push

circleci

Committed by

mark-dce

Commit Message

Remove deprecated calls to `Redis.current`

**ISSUE**
Our logs are being swamped with deprecation messages similar to:
```
`Redis.current` is deprecated and will be removed in 5.0.
  (called from: /Users/mark/.rbenv/versions/2.7.4/lib/ruby/gems/2.7.0/gems
   /hyrax-2.9.6/lib/hyrax/redis_event_store.rb:26:in `instance')
```

**RESOLUTION**
Refactor the redis initilizer and override Hyrax class methods that
call `Redis.current` in favor of a memoized call to `Redis.new`

**REFERENCES**
This commit was heavily influenced by
https://github.com/samvera/hyrax/commit/50cc8f0f

Run Details

17 of 17 new or added lines in 4 files covered. (100.0%)

32 existing lines in 9 files now uncovered.

2836 of 2911 relevant lines covered (97.42%)

52.16 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

97.37

/app/controllers/in_progress_etds_controller.rb

class InProgressEtdsController < ApplicationController
  def new
    authorize! :create, InProgressEtd

    @in_progress_etd = if params[:etd_id]
                         InProgressEtd.find_or_create_by(etd_id: params[:etd_id])
                       else
                         InProgressEtd.find_or_create_by(user_ppid: current_user.ppid)
                       end

    # Now that the record has been created, render the form so student can edit it:
    redirect_to action: :edit, id: @in_progress_etd.id
  end

  # Note: There is no 'create' action because the 'new' action does 'find or create'.

  def edit
    if current_user.nil?
      redirect_to new_user_session_path
    else
      @in_progress_etd = InProgressEtd.find(params[:id])
      authorize! :update, @in_progress_etd
      @in_progress_etd.refresh_from_etd!
      @data = @in_progress_etd.data
    end
  rescue ActiveRecord::RecordNotFound
    redirect_to root_url
  end

  # The Vue.js form uses this action to update the record.
  def update
    @in_progress_etd = InProgressEtd.find(params[:id])
    authorize! :update, @in_progress_etd
    @in_progress_etd.data = prepare_etd_data.to_json

    if @in_progress_etd.save
      @data = @in_progress_etd.data
      render json: { in_progress_etd: @data, lastCompletedStep: current_step(@data), tab_name: tab_name }, status: 200
    else
      render json: { errors: @in_progress_etd.errors.messages }, status: 422
    end
  end

  def destroy
    @in_progress_etd = InProgressEtd.find(params[:id])
    authorize! :update, @in_progress_etd
    @in_progress_etd.destroy

    redirect_to root_url
  end

  private

    def current_step(data)
      saved_data = JSON.parse(data)
      saved_data['currentStep']
    end

    def tab_name
      etd = request.parameters.fetch(:etd)
      etd.fetch(:currentTab, "About Me")
    end

    def prepare_etd_data
      # TODO: strong params
      new_data = request.parameters.fetch(:etd, {})

      # Add the new data to the existing persisted data
      @in_progress_etd.add_data(new_data)
    end
end

1	class InProgressEtdsController < ApplicationController	4✔
2	def new	4✔
3	authorize! :create, InProgressEtd	10✔
4
5	@in_progress_etd = if params[:etd_id]	9✔
UNCOV 6	InProgressEtd.find_or_create_by(etd_id: params[:etd_id])	5✔
7	else
8	InProgressEtd.find_or_create_by(user_ppid: current_user.ppid)	4✔
9	end
10
11	# Now that the record has been created, render the form so student can edit it:
12	redirect_to action: :edit, id: @in_progress_etd.id	9✔
13	end
14
15	# Note: There is no 'create' action because the 'new' action does 'find or create'.
16
17	def edit	4✔
18	if current_user.nil?	13✔
19	redirect_to new_user_session_path	2✔
20	else
21	@in_progress_etd = InProgressEtd.find(params[:id])	11✔
22	authorize! :update, @in_progress_etd	10✔
23	@in_progress_etd.refresh_from_etd!	8✔
24	@data = @in_progress_etd.data	8✔
25	end
26	rescue ActiveRecord::RecordNotFound
27	redirect_to root_url	1✔
28	end
29
30	# The Vue.js form uses this action to update the record.
31	def update	4✔
32	@in_progress_etd = InProgressEtd.find(params[:id])	11✔
33	authorize! :update, @in_progress_etd	11✔
34	@in_progress_etd.data = prepare_etd_data.to_json	9✔
35
36	if @in_progress_etd.save	9✔
37	@data = @in_progress_etd.data	9✔
38	render json: { in_progress_etd: @data, lastCompletedStep: current_step(@data), tab_name: tab_name }, status: 200	9✔
39	else
40	render json: { errors: @in_progress_etd.errors.messages }, status: 422	×
41	end
42	end
43
44	def destroy	4✔
45	@in_progress_etd = InProgressEtd.find(params[:id])	2✔
46	authorize! :update, @in_progress_etd	2✔
47	@in_progress_etd.destroy	1✔
48
49	redirect_to root_url	1✔
50	end
51
52	private	4✔
53
54	def current_step(data)	4✔
55	saved_data = JSON.parse(data)	9✔
56	saved_data['currentStep']	9✔
57	end
58
59	def tab_name	4✔
60	etd = request.parameters.fetch(:etd)	9✔
61	etd.fetch(:currentTab, "About Me")	9✔
62	end
63
64	def prepare_etd_data	4✔
65	# TODO: strong params
66	new_data = request.parameters.fetch(:etd, {})	9✔
67
68	# Add the new data to the existing persisted data
69	@in_progress_etd.add_data(new_data)	9✔
70	end
71	end

curationexperts / laevigata / 814b7cee-7846-4f1b-9594-6ee9d5c461bc

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous