Skip to content

Commit f4a32c6

Browse files
authored
Merge pull request #1450 from PRX/fix/replace_apple_config
Allow the key or apple config to be added if missing after create
2 parents 0960e83 + 1e1dd0c commit f4a32c6

File tree

1 file changed

+13
-8
lines changed

1 file changed

+13
-8
lines changed

app/controllers/feeds_controller.rb

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ def index
99

1010
# GET /feeds/1
1111
def show
12-
@feed.assign_attributes(feed_params)
12+
init_config
1313
authorize @feed
1414
@apple_show_options = get_apple_show_options(@feed)
1515
end
@@ -33,20 +33,15 @@ def get_apple_show_options(feed)
3333

3434
def new_apple
3535
@feed = Feeds::AppleSubscription.new(podcast: @podcast, private: true)
36-
@feed.build_apple_config
37-
@feed.apple_config.build_key
3836
authorize @feed
39-
40-
@feed.assign_attributes(feed_params)
37+
init_config
4138
render "new"
4239
end
4340

4441
def new_megaphone
4542
@feed = Feeds::MegaphoneFeed.new(podcast: @podcast, private: true)
46-
@feed.build_megaphone_config
4743
authorize @feed
48-
49-
@feed.assign_attributes(feed_params)
44+
init_config
5045
render "new"
5146
end
5247

@@ -109,6 +104,16 @@ def destroy
109104

110105
private
111106

107+
def init_config
108+
@feed.assign_attributes(feed_params)
109+
if @feed.is_a? Feeds::AppleSubscription
110+
@feed.build_apple_config unless @feed.apple_config
111+
@feed.apple_config.build_key unless @feed.apple_config.key
112+
elsif @feed.is_a? Feeds::MegaphoneFeed
113+
@feed.megaphone_config || @feed.build_megaphone_config
114+
end
115+
end
116+
112117
def set_podcast
113118
@podcast =
114119
if @feed

0 commit comments

Comments
 (0)