Merge branch 'atomaka/bugfix/card-order' into 'master'

Default scope sucks, fix ordering for cards

`default_scope` is always active.  It can be disabled by creating your ActiveRecord query with `unscoped` but that requires knowing about `default_scope`. Let's remove it and order per use case.

See merge request !22

app/controllers/api/v1/cards_controller.rb

@@ -4,7 +4,7 @@ class Api::V1::CardsController < Api::V1::BaseController
     @cards = Card.includes(:character, :rarity, :type, :leader_skill,
                            :passive_skill, :super_attack, :awaken_type,
                            :dokkan_card, :hp_stat, :atk_stat, :def_stat,
-                           :links)
+                           :links).all.order(:gameid)
 
     render json: @cards, root: false
   end

app/models/card.rb

@@ -35,8 +35,6 @@ class Card < ActiveRecord::Base
   validates :type, presence: true
   validates :awaken_type, presence: true
 
-  default_scope { order(:gameid) }
-
   scope :new_this_week, -> { where('created_at > ?', 1.week.ago) }
 
   def dokkan?

app/views/admin/cards/index.html.slim

@@ -11,12 +11,14 @@
         tr
           th Name
           th Title
+          th Updated
           th Actions
       tbody
         - @cards.each do |card|
           tr
             td= card.name
             td= card.title
+            td= card.updated_at
             td
               - if policy(:card).edit?
                 = link_to glyph('edit', classes: 'control-icon'), edit_admin_card_path(card)