atzorvas/ccradio

Missing top-level class documentation comment.
class StreamsController < ApplicationController
  before_action :set_stream,
Align the parameters of a method call if they span more than one line.
Use `%i` or `%I` for an array of symbols.
    only: [:show, :edit, :update, :destroy, :playlist, :current_song]
  before_action :authenticate_admin,
Align the parameters of a method call if they span more than one line.
Use `%i` or `%I` for an array of symbols.
    except: [:show, :index, :playlist, :current_song, :subscription]
  include Tubesock::Hijack
 
  # GET /streams
  # GET /streams.json
  def index
    @streams = Stream.all
  end
 
  # GET /streams/1
  # GET /streams/1.json
Put empty method definitions on a single line.
  def show
  end
 
  # GET /streams/new
  def new
    @stream = Stream.new
  end
 
  # GET /streams/1/edit
Put empty method definitions on a single line.
  def edit
  end
 
  # POST /streams
  # POST /streams.json
  def create
    @stream = Stream.new(stream_params)
 
    respond_to do |format|
      if @stream.save
Line is too long. [87/80]
        format.html { redirect_to @stream, notice: 'Stream was successfully created.' }
        format.json { render :show, status: :created, location: @stream }
      else
        format.html { render :new }
Line is too long. [82/80]
        format.json { render json: @stream.errors, status: :unprocessable_entity }
      end
    end
  end
 
  # PATCH/PUT /streams/1
  # PATCH/PUT /streams/1.json
  def update
    respond_to do |format|
      if @stream.update(stream_params)
Line is too long. [87/80]
        format.html { redirect_to @stream, notice: 'Stream was successfully updated.' }
        format.json { render :show, status: :ok, location: @stream }
      else
        format.html { render :edit }
Line is too long. [82/80]
        format.json { render json: @stream.errors, status: :unprocessable_entity }
      end
    end
  end
 
  # DELETE /streams/1
  # DELETE /streams/1.json
  def destroy
    @stream.destroy
    respond_to do |format|
Line is too long. [91/80]
      format.html { redirect_to streams_url, notice: 'Stream was successfully destroyed.' }
      format.json { head :no_content }
    end
  end
 
  def playlist
    respond_to do |format|
Avoid using `{...}` for multi-line blocks.
      format.json {
        render json: {
          genre: @stream.title,
          url: @stream.url_play,
Prefer single-quoted strings when you don't need string interpolation or special symbols.
          history: @stream.playlist_items.order("created_at DESC")
        }
      }
      format.html { redirect_to @stream }
    end
  end
 
  def current_song
    item = @stream.playlist_items.last
    respond_to do |format|
Line is too long. [83/80]
      format.json { render json: { song: item.song, created_at: item.created_at } }
      format.html { redirect_to @stream }
    end
  end
 
Method has too many lines. [15/10]
  def subscription
    hijack do |sock|
      redis_thread = Thread.new do
Prefer single-quoted strings when you don't need string interpolation or special symbols.
        Redis.new.subscribe "songs" do |on|
          on.message do |_channel, song|
            sock.send_data song
          end
        end
      end
 
      sock.onmessage do |m|
Prefer single-quoted strings when you don't need string interpolation or special symbols.
        Redis.new.publish "songs", m
      end
 
      sock.onclose do
        redis_thread.kill
      end
    end
  end
 
Keep a blank line before and after `private`.
  private
  # Use callbacks to share common setup or constraints between actions.
  def set_stream
    @stream = Stream.find(params[:id] || params[:stream_id])
  end
 
  # Never trust parameters from the scary internet.
  def stream_params
    params.require(:stream).permit(:title, :server, :mount)
  end
end