FIXUP fragment caching issues

app/controllers/conferences_controller.rb

@@ -12,7 +12,6 @@ class ConferencesController < ApplicationController
     @conference = Conference.unscoped.eager_load(
       :organization,
       :splashpage,
-      :venue,
       :registration_period,
       :tickets,
       :confirmed_tracks,
@@ -26,6 +25,7 @@ class ConferencesController < ApplicationController
       :sponsors,
       :call_for_sponsors,
       :contact,
+      venue:              [:commercial],
       highlighted_events: [:speakers],
       sponsorship_levels: [:sponsors]
     ).order(

app/views/conferences/_booths.haml

@@ -13,5 +13,3 @@
         - else
           .caption
             %h3.text-center= booth.title
-    - content_for :modals do
-      = render 'modal_description', object: booth

app/views/conferences/_modal_description.haml

@@ -1,21 +1,22 @@
 - content_for :modals do
-  .modal.fade{ id: "modal-#{object.class.name.downcase}-#{object.id}" }
+  - cache [object, '#modal'] do
-    .modal-dialog
+    .modal.fade{ id: "modal-#{object.class.name.downcase}-#{object.id}" }
-      .modal-content
+      .modal-dialog
-        .modal-header
+        .modal-content
-          %button.close{ data: { dismiss: 'modal' } }
+          .modal-header
-            %i.fa.fa-close
+            %button.close{ data: { dismiss: 'modal' } }
-          %h3.modal-title
+              %i.fa.fa-close
-            = object.try(:title) || object.try(:name)
+            %h3.modal-title
-        .modal-body
+              = object.try(:title) || object.try(:name)
-          %p.logo
+          .modal-body
-            = link_to object.website_url || '#', target: '_blank' do
+            %p.logo
-              - img_classes = ['img-responsive center-block']
+              = link_to object.website_url || '#', target: '_blank' do
-              - if object.is_a? Sponsor
+                - img_classes = ['img-responsive center-block']
-                - img_classes << ['img-sponsor',
+                - if object.is_a? Sponsor
-                  "img-sponsor-#{object.sponsorship_level.position}"]
+                  - img_classes << ['img-sponsor',
-              = image_tag get_logo(object), class: img_classes
+                    "img-sponsor-#{object.sponsorship_level.position}"]
-          %p.description
+                = image_tag get_logo(object), class: img_classes
-            = object.description
+            %p.description
-        .modal-footer
+              = object.description
-          = link_to nil, object.website_url, target: '_blank'
+          .modal-footer
+            = link_to nil, object.website_url, target: '_blank'

app/views/conferences/_program.haml

@@ -31,3 +31,7 @@
               = link_to(conference_schedule_path(conference.short_title),
                 class: 'btn btn-success btn-lg') do
                 Full Schedule
+
+- conference.confirmed_booths.each do |booth|
+  - content_for :modals do
+    = render 'modal_description', object: booth

app/views/conferences/_sponsors.haml

@@ -21,8 +21,6 @@
                     class: ['img-responsive', 'img-sponsor',
                     "img-sponsor-#{sponsorship_level.position}"],
                     title: sponsor.name
-              - content_for :modals do
-                = render 'modal_description', object: sponsor
 
       - if conference.call_for_sponsors.try(:open?)
         .row
@@ -31,3 +29,7 @@
               Want to sponsor?
               = link_to(sponsorship_mailto(conference)) do
                 Contact us!
+
+- conference.sponsors.each do |sponsor|
+  - content_for :modals do
+    = render 'modal_description', object: sponsor

app/views/conferences/_venue.haml

@@ -2,11 +2,11 @@
   %li
     %a.smoothscroll{ href: '#venue' } Venue
 
-- cache [conference.venue, conference.venue.commercial, '#splash#venue'] do
+%section#venue
-  %section#venue
+  - if conference.venue.location?
-    - if conference.venue.location?
+    = render '/conferences/venue_map', conference: conference
-      = render '/conferences/venue_map', conference: conference
+  - else
-    - else
+    - cache [conference.venue, conference.venue.commercial, '#splash#venue'] do
       .container
         .row
           .col-md-6

app/views/layouts/application.html.haml

@@ -43,5 +43,5 @@
             You can run, copy, distribute, study, change and improve it.
             The source code and the developers are on
             =link_to "github.", "https://github.com/openSUSE/osem"
-    = content_for(:script_body)
+    = yield :script_body
     = piwik_tracking_tag