diff --git a/app/assets/javascripts/app.js.erb b/app/assets/javascripts/app.js.erb index a40c034f4..dabc9e83e 100644 --- a/app/assets/javascripts/app.js.erb +++ b/app/assets/javascripts/app.js.erb @@ -125,6 +125,20 @@ config(['$httpProvider', 'AuthProvider', "growlProvider", "unsavedWarningsConfig // see https://github.com/revolunet/angular-google-analytics#automatic-page-view-tracking Analytics.pageView(); + + /** + * This helper method builds and return an array contaning every integers between + * the provided start and end. + * @param start {number} + * @param end {number} + * @return {Array} [start .. end] + */ + $rootScope.intArray = function(start, end) { + var arr = []; + for (var i = start; i < end; i++) { arr.push(i); } + return arr; + }; + }]).constant('angularMomentConfig', { timezone: Fablab.timezone }); diff --git a/app/assets/javascripts/controllers/projects.coffee.erb b/app/assets/javascripts/controllers/projects.coffee.erb index b9a2f2131..5b2953c3d 100644 --- a/app/assets/javascripts/controllers/projects.coffee.erb +++ b/app/assets/javascripts/controllers/projects.coffee.erb @@ -122,7 +122,7 @@ class ProjectsController # This will create a single new empty entry into the project's steps list. ## $scope.addStep = -> - $scope.project.project_steps_attributes.push {} + $scope.project.project_steps_attributes.push { step_nb: $scope.project.project_steps_attributes.length + 1 } diff --git a/app/assets/templates/projects/_form.html.erb b/app/assets/templates/projects/_form.html.erb index 4dbecba73..7e4d77e9f 100644 --- a/app/assets/templates/projects/_form.html.erb +++ b/app/assets/templates/projects/_form.html.erb @@ -74,9 +74,18 @@
- {{ 'step_N' | translate:{ INDEX:$index+1 } }}/{{project.project_steps_attributes.length}} +
+ + +
+ +
-

{{ 'step_N' | translate:{INDEX:$index+1} }} : {{step.title}}

+

{{ 'step_N' | translate:{INDEX:step.step_nb} }} : {{step.title}}

{{step.title}} diff --git a/app/controllers/api/projects_controller.rb b/app/controllers/api/projects_controller.rb index f78ffe555..2a6d4fffe 100644 --- a/app/controllers/api/projects_controller.rb +++ b/app/controllers/api/projects_controller.rb @@ -66,7 +66,7 @@ class API::ProjectsController < API::ApiController params.require(:project).permit(:name, :description, :tags, :machine_ids, :component_ids, :theme_ids, :licence_id, :author_id, :licence_id, :state, user_ids: [], machine_ids: [], component_ids: [], theme_ids: [], project_image_attributes: [:attachment], project_caos_attributes: [:id, :attachment, :_destroy], - project_steps_attributes: [:id, :description, :title, :_destroy, + project_steps_attributes: [:id, :description, :title, :_destroy, :step_nb, :project_step_image_attributes => :attachment]) end end diff --git a/app/views/api/projects/show.json.jbuilder b/app/views/api/projects/show.json.jbuilder index 495a9c42c..ab13940d7 100644 --- a/app/views/api/projects/show.json.jbuilder +++ b/app/views/api/projects/show.json.jbuilder @@ -46,12 +46,13 @@ json.project_users @project.project_users do |pu| json.slug pu.user.slug json.is_valid pu.is_valid end -json.project_steps_attributes @project.project_steps.order('project_steps.created_at ASC') do |s| +json.project_steps_attributes @project.project_steps.order('project_steps.step_nb ASC') do |s| json.id s.id json.description s.description json.title s.title json.project_step_image s.project_step_image.attachment_identifier if s.project_step_image json.project_step_image_url s.project_step_image.attachment.medium.url if s.project_step_image + json.step_nb s.step_nb end json.state @project.state json.licence do diff --git a/db/migrate/20160602075531_add_step_nb_to_project_step.rb b/db/migrate/20160602075531_add_step_nb_to_project_step.rb new file mode 100644 index 000000000..2f41200f8 --- /dev/null +++ b/db/migrate/20160602075531_add_step_nb_to_project_step.rb @@ -0,0 +1,19 @@ +class AddStepNbToProjectStep < ActiveRecord::Migration + def up + add_column :project_steps, :step_nb, :integer + execute 'UPDATE project_steps + SET step_nb = subquery.index + FROM ( + SELECT + id, project_id, created_at, + row_number() OVER (PARTITION BY project_id) AS index + FROM project_steps + ORDER BY created_at + ) AS subquery + WHERE project_steps.id = subquery.id;' + end + + def down + remove_column :project_steps, :step_nb + end +end diff --git a/db/schema.rb b/db/schema.rb index 1085aac04..bd88b76bb 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -11,7 +11,7 @@ # # It's strongly recommended that you check this file into your version control system. -ActiveRecord::Schema.define(version: 20160526102307) do +ActiveRecord::Schema.define(version: 20160602075531) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -337,6 +337,7 @@ ActiveRecord::Schema.define(version: 20160526102307) do t.datetime "created_at" t.datetime "updated_at" t.string "title", limit: 255 + t.integer "step_nb" end add_index "project_steps", ["project_id"], name: "index_project_steps_on_project_id", using: :btree