Merge 2905c8858c into 301ad1c722

app/controllers/conference_controller.rb

@@ -15,6 +15,11 @@ class ConferenceController < ApplicationController
     @rooms = @conference.venue.rooms if @conference.venue
     @events = @conference.program.events
     @dates = @conference.start_date..@conference.end_date
+    @start_hour = 9
+    @end_hour = 19
+    @intervals = (@end_hour - @start_hour) * 60 / EventType::LENGTH_STEP
+    @width = 95 / @intervals
+    @step_minutes = EventType::LENGTH_STEP.minutes
 
     if @dates == Date.current
       @today = Date.current.strftime('%Y-%m-%d')

app/models/conference.rb

@@ -81,6 +81,16 @@ class Conference < ActiveRecord::Base
     result
   end
 
+  ##
+  # Check if there is any event in the program scheduled.
+  #
+  # ====Returns
+  # * +false+ ->  If there aren't schedules events
+  # * +true+ -> If there are schedules events
+  def scheduled?
+    self.program.events.where.not(start_time: nil).any?
+  end
+
   ##
   # Checks if the user is registered to the conference
   #

app/models/event.rb

@@ -223,6 +223,13 @@ class Event < ActiveRecord::Base
     (100 * result.values.count(true) / result.values.compact.count).to_s
   end
 
+  ##
+  # Returns the number of columns this event should span in the schedule
+  #
+  def span
+    self.event_type.length / EventType::LENGTH_STEP
+  end
+
   private
 
   ##

app/models/event_type.rb

@@ -8,4 +8,7 @@ class EventType < ActiveRecord::Base
   validates :maximum_abstract_length, presence: true
 
   alias_attribute :name, :title
+
+  # If LENGTH_STEP > 60 the public schedule must be changed
+  LENGTH_STEP = 15
 end

app/models/program.rb

@@ -26,10 +26,6 @@ class Program < ActiveRecord::Base
       where(state: :confirmed)
     end
 
-    def scheduled
-      where.not(start_time: nil)
-    end
-
     def highlights
       where(state: :confirmed, is_highlight: true)
     end

app/views/conference/_schedule.html.erb

@@ -13,51 +13,36 @@
         <div class="tab-pane <%= 'active' if @dates.first == date %>" id ="<%= date %>" >
           <table class="table table-bordered" id="schedule" width="100%">
             <tr>
-              <th>Time</th>
-              <% @rooms.each do |room| %>
-                <th><%= room.name %></th>
+              <th>Rooms</th>
+              <% (@start_hour..@end_hour-1).each do |i| %>
+                  <th colspan="4"><%= "#{i}:00" %></th>
               <% end %>
             </tr>
-
-                <% span = {} %>
             <% @rooms.each do |room| %>
-                  <% span[room.id] = 1 %>
-                <% end %>
-
-            <% conf_start = DateTime.parse("#{date} 09:00") %>
-            <% conf_end = DateTime.parse("#{date} 19:00") %>
-
-            <% while conf_start < conf_end %>
-              <%-
-                start_hour = conf_start.hour
-                start_min = conf_start.min %>
-
+              <% span = 1 %>
               <tr>
-                <td style="width:5%">
-                  <%= conf_start.strftime("%H:%M") %>
-                </td>
-
-                <% @rooms.each do |room| %>
-                  <% events_for_date = @events.find_all {|e| e.room_id == room.id && e.start_time && e.start_time.to_date == date } %>
-                  <% event = events_for_date.find_all{|e| e.start_time.hour == start_hour && e.start_time.min == start_min } %>
-                  <% if !event.empty? %>
+                <td><%= room.name %></td>
+                  <% events_for_date = room.events.find_all{ |e| e.start_time.to_date == date } %>
+                  <% start_time = DateTime.parse("#{date} #{@start_hour}:00") %>
+                  <% (1..@intervals).each do |i| %>
+                    <% events_for_time = events_for_date.find_all{|e| e.start_time == start_time} %>
+                    <% if !events_for_time.empty? %>
+                      <% event = events_for_time[0] %>
                       <!-- There is an event, calculate the span and show it -->
-                    <% span[room.id] = event[0].event_type.length / 15 %>
-                    <%= render partial: 'schedule_item', locals: {event: event, room: room, span: span} %>
-                    <% span[room.id] = span[room.id] + 1 %>
-                    <% elsif  span[room.id] > 1 %>
-                      <!-- span is <%= span[room.id] %> and bigger than 1, substract 1 from span -->
-                      <% span[room.id] = ( span[room.id] - 1 ) if span[room.id] > 1 %>
+                      <% span = event.span %>
+                      <%= render partial: 'schedule_item', locals: {event: event, span: span, width: @width} %>
+                      <% span += 1 %>
+                    <% elsif  span > 1 %>
+                      <% span -= 1 %>
                     <% end %>
-                    <% if span[room.id] == 1 %>
+                    <% if span == 1 %>
                       <!-- span equals 1 show an empty td -->
-                      <td style="width: <%= 95 / @rooms.length %>%">
-                      </td>
+                      <td style="width: <%= @width %>%"></td>
+                    <% end %>
+                    <% start_time += @step_minutes %>
                   <% end %>
-                <%end %>
               </tr>
-              <% conf_start += 15.minutes %>
-            <%end %>
+            <% end %>
           </table>
         </div>
       <% end %>

app/views/conference/_schedule_item.html.haml

@@ -1,24 +1,11 @@
-%td.event{ style: "width: #{95 / @rooms.length}%; cursor:pointer", |
-           rowspan: span[room.id], |
+%td.event{ style: "width: #{width * span}%; cursor:pointer", |
+           colspan: span, |
            role: "button" } |
-  %a.unstyled-link{href: url_for(conference_program_proposal_path(@conference.short_title, event[0].id))}
-    - if speaker = event[0].speakers.first
+  %a.unstyled-link{href: url_for(conference_program_proposal_path(@conference.short_title, event.id))}
+    %h5
+      = event.title
+    - if speaker = event.speakers.first
       = image_tag speaker.gravatar_url, :class => "img-circle pull-right", |
                                          :alt => speaker.name, |
                                          :title => speaker.name, |
-                                         :style => "padding:8px;" |
-    %h5
-      %span.fa.fa-comment
-      = event[0].title
-      - unless event[0].subtitle.blank?
-        %span.schedule-subtitle
-          = event[0].subtitle
-    %span.schedule-speaker
-      %span.fa.fa-user
-      = speaker.name
-    - if event[0].track
-      %span.schedule-track
-        %span.fa.fa-road
-        %span.label{ style: "background-color: #{event[0].track.color}" }
-          = event[0].track.name
-
+                                         :style => "padding:2px 0px;" |

app/views/conference/schedule.html.erb

@@ -1,5 +1,5 @@
 <div class="container">
-  <% if @conference.program.events.scheduled.any? %>
+  <% if @conference.scheduled? %>
       <div role="tabpanel">
         <!-- Nav tabs -->
         <ul class="nav nav-tabs" role="tablist">