From f6ba160db632f851af360cd9d5f520b54e7779af Mon Sep 17 00:00:00 2001
From: Kyle Drake <kyle@kyledrake.net>
Date: Tue, 23 Dec 2014 09:23:26 -0800
Subject: [PATCH] disable surf mode until we can figure out what to do about
 the clickjacking restrictions

---
 models/site.rb      | 5 +++++
 views/browse.erb    | 4 ++--
 views/dashboard.erb | 2 +-
 views/home.erb      | 4 ++--
 views/site.erb      | 4 ++--
 5 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/models/site.rb b/models/site.rb
index cf3c3c36..21b52e15 100644
--- a/models/site.rb
+++ b/models/site.rb
@@ -938,6 +938,11 @@ class Site < Sequel::Model
     !domain.empty? ? domain : "#{username}.neocities.org"
   end
 
+  def uri
+    # Switch-over for when SSL defaulting is ready
+    "http://#{host}"
+  end
+
   def title
     if values[:title].nil? || values[:title].empty?
       !domain.nil? && !domain.empty? ? domain : "#{username}.neocities.org"
diff --git a/views/browse.erb b/views/browse.erb
index 9a7babb8..29b17357 100644
--- a/views/browse.erb
+++ b/views/browse.erb
@@ -63,13 +63,13 @@
   <ul class="row website-Gallery content int-Gall">
     <% @sites.each_with_index do |site,i| %>
       <li>
-        <a href="" class="neo-Screen-Shot" title="<%= site.title %>" onclick="surf(<%= ((@current_page-1)*Site::BROWSE_PAGINATION_LENGTH)+i+1 %>); return false">
+        <a href="<%= site.uri %>" class="neo-Screen-Shot" title="<%= site.title %>" onclick="return true; surf(<%= ((@current_page-1)*Site::BROWSE_PAGINATION_LENGTH)+i+1 %>); return false">
           <span class="img-Holder" style="background:url(<%= site.screenshot_url('index.html', '540x405') %>) no-repeat;">
             <img src="/img/placeholder.png" alt="<%= site.title %>" />
           </span>
         </a>
         <div class="title">
-          <a href="" title="<%= site.title %>" onclick="surf(<%= i+1 %>); return false"><%= site.title.shorten(30) %></a>
+          <a href="<%= site.uri %>" title="<%= site.title %>" onclick="return true; surf(<%= i+1 %>); return false"><%= site.title.shorten(30) %></a>
         </div>
         <div class="site-info">
           <div class="username">
diff --git a/views/dashboard.erb b/views/dashboard.erb
index a4d86ec1..d1d0206a 100644
--- a/views/dashboard.erb
+++ b/views/dashboard.erb
@@ -22,7 +22,7 @@
     <div class="col col-50 signup-Area">
       <div class="signup-Form">
   	  <fieldset class="content">
-        <a href="/surf/<%= current_site.username %>" class="screenshot dashboard" style="background-image:url(<%= current_site.screenshot_url('index.html', '540x405') %>);"></a>
+        <a href="<%= current_site.uri %>" class="screenshot dashboard" style="background-image:url(<%= current_site.screenshot_url('index.html', '540x405') %>);"></a>
 	    </fieldset>
       </div>
     </div>
diff --git a/views/home.erb b/views/home.erb
index 31df3c74..cb9961e8 100644
--- a/views/home.erb
+++ b/views/home.erb
@@ -51,13 +51,13 @@
           <ul class="row website-Gallery content int-Gall suggestions">
           <% @suggestions.each do |suggested_site| %>
             <li>
-              <a href="/surf/<%= suggested_site.username %>" class="neo-Screen-Shot" title="<%= suggested_site.title %>">
+              <a href="<%= suggested_site.uri %>" class="neo-Screen-Shot" title="<%= suggested_site.title %>">
                 <span class="img-Holder" style="background:url(<%= suggested_site.screenshot_url('index.html', '540x405') %>) no-repeat;">
                   <img src="/img/placeholder.png" alt="<%= suggested_site.title %>" />
                 </span>
               </a>
               <div class="title">
-                <a href="/surf/<%= suggested_site.username %>" title="<%= suggested_site.title %>"><%= suggested_site.title.shorten(30) %></a>
+                <a href="<%= suggested_site.uri %>" title="<%= suggested_site.title %>"><%= suggested_site.title.shorten(30) %></a>
               </div>
               <div class="site-info">
                 <div class="username">
diff --git a/views/site.erb b/views/site.erb
index e936fc56..560eeb5e 100644
--- a/views/site.erb
+++ b/views/site.erb
@@ -14,13 +14,13 @@
     <div class="col col-50 signup-Area large">
       <div class="signup-Form">
   	  <fieldset class="content">
-        <a href="/surf/<%= site.username %>" class="screenshot" style="background-image:url(<%= site.screenshot_url('index.html', '540x405') %>);"></a>
+        <a href="<%= site.uri %>" class="screenshot" style="background-image:url(<%= site.screenshot_url('index.html', '540x405') %>);"></a>
 	    </fieldset>
       </div>
     </div>
     <div class="col col-50 profile-info">
       <h2 class="eps title-with-badge"><span><%= site.title %></span> <% if site.supporter? %><a href="/plan" class="supporter-badge" title="Neocities Supporter"></a> <% end %></h2>
-      <p class="site-url"><a href="/surf/<%= site.username %>"><%= site.host %></a></p>
+      <p class="site-url"><a href="<%= site.uri %>"><%= site.host %></a></p>
       <div class="stats">
         <div class="stat"><strong><%= site.views.format_large_number %></strong> <span>views</span></div>
         <% follows_count = site.follows_dataset.count %>