Merge pull request #1 from dsjoerg/master

fix document.domain issue with IP hostnames

Author: nickelsen

app/assets/javascripts/setdocdomain.js

@@ -1,4 +1,8 @@
 // Proxino.log("gg.js domain was " + document.domain);
 // weird, was ggtracker.com even though script was served via CDN
-document.domain = /(\w+)(.\w+)?$/.exec(location.hostname)[0];
+if (location.hostname.match(/^[0-9.]*$/) == null) {
+    // only set document.domain if our location is a hostname like ggtracker.com or
+    // ggtracker.test.   But if it's an IP address, dont bother.
+    document.domain = /(\w+)(.\w+)?$/.exec(location.hostname)[0];
+}
 // Proxino.log("now gg.js domain is " + document.domain);

app/views/players/show.html.erb

@@ -11,7 +11,6 @@
         <h2><a href="/players/<%= @identity.id %>/<%= @identity.name %>"><%= @identity.name %></a><span ng-show="<%= @prolevel %> > 0" class="djcloak icon icon-star-white-32"></span><!--<span class="tiny prolabel">PRO</span>--></h2>
         <div class="links">
           <%= link_to('bnet', @identity.profile_url, {:class => 'bnet'}) if @identity.profile_url.present? %>
-          <%= link_to('sc2ranks', @identity.sc2ranks_url) if @identity.sc2ranks_url.present? %>
           <% if controller.superuser? %>
             <%= link_to('+1Month', :controller => :players, :action => :give_one_month_ggtracker_pro) %>
           <% end %>