Remove sequence number from performer.

lib/runosaari/registration.ex

@@ -22,7 +22,7 @@ defmodule Runosaari.Registration do
   end
 
   @doc """
-  Returns the :seqnum sorted list of confirmed performers.
+  Returns the :name sorted list of confirmed performers.
 
   ## Examples
 
@@ -33,7 +33,7 @@ defmodule Runosaari.Registration do
   def list_sorted_confirmed_performers do
     Performer
     |> where([p], p.confirmed == true)
-    |> order_by(:seqnum)
+    |> order_by(:name)
     |> Repo.all()
   end
 

lib/runosaari/registration/performer.ex

@@ -9,7 +9,6 @@ defmodule Runosaari.Registration.Performer do
     field :paragraph3, :string
     field :paragraph4, :string
     field :paragraph5, :string
-    field :seqnum, :integer, default: 999
     field :photo_path, :string
     field :confirmed, :boolean, default: false
 
@@ -21,7 +20,6 @@ defmodule Runosaari.Registration.Performer do
     performer
     |> cast(attrs, [
       :name,
-      :seqnum,
       :paragraph1,
       :paragraph2,
       :paragraph3,
@@ -32,7 +30,6 @@ defmodule Runosaari.Registration.Performer do
     ])
     |> validate_required([
       :name,
-      :seqnum,
       :photo_path,
       :confirmed
     ])

lib/runosaari_web/controllers/performer_controller.ex

@@ -71,7 +71,7 @@ defmodule RunosaariWeb.PerformerController do
       {:ok, performer} ->
         conn
         |> put_flash(:info, "Esiintyjän tiedot päivitetty.")
-        |> redirect(to: Routes.performer_path(conn, :show, performer))
+        |> redirect(to: Routes.admin_performer_path(conn, :admin))
 
       {:error, %Ecto.Changeset{} = changeset} ->
         render(conn, "edit.html", performer: performer, changeset: changeset)

lib/runosaari_web/templates/performer/admin.html.eex

@@ -14,7 +14,6 @@
         <tr>
           <td><%= performer.name %></td>
           <td><%= performer.confirmed %></td>
-          <td><%= performer.seqnum %></td>
 
           <td class="actions">
             <span><%= link "Esiintyjäsivu", to: Routes.performer_path(@conn, :show, performer) %></span>

lib/runosaari_web/templates/performer/form.html.eex

@@ -9,7 +9,6 @@
   <%= text_input form, :name %>
   <%= error_tag form, :name %>
 
-  <hr />
   <%= label form, :paragraph1, "Kappale 1 (max 2000 merkkiä)" %>
   <%= textarea form, :paragraph1 %>
   <%= error_tag form, :paragraph1 %>
@@ -30,10 +29,6 @@
   <%= textarea form, :paragraph5 %>
   <%= error_tag form, :paragraph5 %>
 
-  <%= label form, :seqnum, "Prioriteetti (1 on korkein)" %>
-  <%= number_input form, :seqnum %>
-  <%= error_tag form, :seqnum %>
-
   <%= label form, :photo, "Esiintyjän kuva (muokatessa ei vaadi uudelleen lisäys vaikka tyhjä)" %>
   <%= file_input form, :photo %>
   <%= error_tag form, :photo %>

priv/repo/migrations/20210602164721_remove_performer_seqnum.exs

@@ -0,0 +1,9 @@
+defmodule Runosaari.Repo.Migrations.RemovePerformerSeqnum do
+  use Ecto.Migration
+
+  def change do
+    alter table("performers") do
+      remove :seqnum
+    end
+  end
+end

test/runosaari/registration_test.exs

@@ -9,20 +9,18 @@ defmodule Runosaari.RegistrationTest do
     @create_attrs %{
       name: "some name",
       desc: "some description",
-      confirmed: false,
+      confirmed: false
-      seqnum: 1
     }
 
     @update_attrs %{
       name: "some updated name",
       desc: "some updated description",
-      confirmed: true,
+      confirmed: true
-      seqnum: 2
     }
 
-    @valid_attrs %{name: "some name", desc: "some description", confirmed: false, seqnum: 1}
+    @valid_attrs %{name: "some name", desc: "some description", confirmed: false}
 
-    @invalid_attrs %{name: nil, desc: nil, confirmed: nil, seqnum: nil}
+    @invalid_attrs %{name: nil, desc: nil, confirmed: nil}
 
     def performer_fixture(attrs \\ %{}) do
       {:ok, performer} =
@@ -48,7 +46,6 @@ defmodule Runosaari.RegistrationTest do
       assert performer.name == "some name"
       assert performer.desc == "some description"
       assert performer.confirmed == false
-      assert performer.seqnum == 1
     end
 
     test "create_performer/1 with invalid data returns error changeset" do
@@ -64,7 +61,6 @@ defmodule Runosaari.RegistrationTest do
       assert performer.name == "some updated name"
       assert performer.desc == "some updated description"
       assert performer.confirmed == true
-      assert performer.seqnum == 2
     end
 
     test "update_performer/2 with invalid data returns error changeset" do