Emi Matchu
a385a5b962
Previously, when opening the pose picker and looking at Styles, the Cybunny options were sorted like this: - Default - Celebratory 25th Anniversary - Festive Christmas - Nostalgic Baby - Nostalgic Blue - Nostalgic Christmas - Nostalgic Darigan - Nostalgic Faerie - Nostalgic Grey - Nostalgic Maraquan - Nostalgic Mutant - Nostalgic Plushie - Nostalgic Robot - Nostalgic Royalboy - Nostalgic Royalgirl - Nostalgic Snow - Nostalgic Tyrannian - Prismatic Cocoa: Festive Christmas - Prismatic Cocoa: Nostalgic Christmas - Prismatic Tinsel: Festive Christmas - Prismatic Tinsel: Nostalgic Christmas - Spooky Halloween Now, they're sorted like this: - Default - Celebratory 25th Anniversary - Nostalgic Baby - Nostalgic Blue - Festive Christmas - Prismatic Cocoa: Festive Christmas - Prismatic Tinsel: Festive Christmas - Nostalgic Christmas - Prismatic Cocoa: Nostalgic Christmas - Prismatic Tinsel: Nostalgic Christmas - Nostalgic Darigan - Nostalgic Faerie - Nostalgic Grey - Spooky Halloween - Nostalgic Maraquan - Nostalgic Mutant - Nostalgic Plushie - Nostalgic Robot - Nostalgic Royalboy - Nostalgic Royalgirl - Nostalgic Snow - Nostalgic Tyrannian Note especially the Christmas case, which is all together now! I think it's also more in line with people's expectations for Halloween to be alphabetically among the rest, instead of being at the bottom for being "Spooky". There's enough styles now that I'm starting to wonder if there's other UI affordances worth having here, like e.g. only showing (or at least prioritizing) styles that match the chosen color? But I don't want to mislead people about compatibility, either.
98 lines
2.5 KiB
Ruby
98 lines
2.5 KiB
Ruby
class AltStylesController < ApplicationController
|
|
before_action :support_staff_only, except: [:index]
|
|
|
|
def index
|
|
@all_series_names = AltStyle.all_series_names
|
|
@all_color_names = AltStyle.all_supported_colors.map(&:human_name).sort
|
|
@all_species_names = AltStyle.all_supported_species.map(&:human_name).sort
|
|
|
|
@series_name = params[:series]
|
|
@color = find_color
|
|
@species = find_species
|
|
|
|
@alt_styles = AltStyle.includes(:color, :species, :swf_assets)
|
|
@alt_styles.where!(series_name: @series_name) if @series_name.present?
|
|
@alt_styles.merge!(@color.alt_styles) if @color
|
|
@alt_styles.merge!(@species.alt_styles) if @species
|
|
|
|
# We're using the HTML5 image for our preview, so make sure we have all the
|
|
# manifests ready!
|
|
SwfAsset.preload_manifests @alt_styles.map(&:swf_assets).flatten
|
|
|
|
respond_to do |format|
|
|
format.html {
|
|
@alt_styles = @alt_styles.
|
|
by_creation_date.order(:color_id, :species_id, :series_name).
|
|
paginate(page: params[:page], per_page: 30)
|
|
render
|
|
}
|
|
format.json {
|
|
@alt_styles = @alt_styles.includes(swf_assets: [:zone]).by_name_grouped
|
|
render json: @alt_styles.as_json(
|
|
only: [:id, :species_id, :color_id, :body_id, :series_name,
|
|
:adjective_name, :thumbnail_url],
|
|
include: {
|
|
swf_assets: {
|
|
only: [:id, :body_id],
|
|
include: [:zone],
|
|
methods: [:urls, :known_glitches],
|
|
}
|
|
},
|
|
methods: [:series_name, :adjective_name, :thumbnail_url],
|
|
)
|
|
}
|
|
end
|
|
end
|
|
|
|
def edit
|
|
@alt_style = AltStyle.find params[:id]
|
|
end
|
|
|
|
def update
|
|
@alt_style = AltStyle.find params[:id]
|
|
|
|
if @alt_style.update(alt_style_params)
|
|
flash[:notice] = "\"#{@alt_style.full_name}\" successfully saved!"
|
|
redirect_to destination_after_save
|
|
else
|
|
render action: :edit, status: :bad_request
|
|
end
|
|
end
|
|
|
|
protected
|
|
|
|
def alt_style_params
|
|
params.require(:alt_style).permit(:real_series_name, :thumbnail_url)
|
|
end
|
|
|
|
def find_color
|
|
if params[:color]
|
|
Color.find_by(name: params[:color])
|
|
end
|
|
end
|
|
|
|
def find_species
|
|
if params[:species_id]
|
|
Species.find_by(id: params[:species_id])
|
|
elsif params[:species]
|
|
Species.find_by(name: params[:species])
|
|
end
|
|
end
|
|
|
|
def destination_after_save
|
|
if params[:next] == "unlabeled-style"
|
|
next_unlabeled_style_path
|
|
else
|
|
alt_styles_path
|
|
end
|
|
end
|
|
|
|
def next_unlabeled_style_path
|
|
unlabeled_style = AltStyle.unlabeled.newest.first
|
|
if unlabeled_style
|
|
edit_alt_style_path(unlabeled_style, next: "unlabeled-style")
|
|
else
|
|
alt_styles_path
|
|
end
|
|
end
|
|
end
|