Merge branch 'hotfix/0.43.1.0'

app/helpers/analytics_helper.rb

@@ -46,7 +46,7 @@ module AnalyticsHelper
   #   # => "if (ga) { ga('send','event','test','Embedded',window.location.href,1) };"
   #
   # Returns a string of a GA JavaScript function to drop into an :onclick handler
-  def track_analytics_event(event_category, event_action, options={})
+  def legacy_track_analytics_event(event_category, event_action, options={})
     begin
       value = if options[:value].nil?
         1
@@ -66,6 +66,10 @@ def track_analytics_event(event_category, event_action, options={})
     "if (ga) { ga('send','event',#{event_args}) };"
   end
 
+  def track_analytics_event(event_category, event_action, options={})
+    # noop, not implemented for GA4 yet
+  end
+
   private
 
   # Private: Format the event label by wrapping in single quotes if it is

app/views/alaveteli_pro/dashboard/_ga_events.html.erb

@@ -1,5 +1,7 @@
 <% if flash[:new_pro_user] %>
   <% content_for :ga_pageview do %>
-    ga('send', 'pageview', '/alaveteli_pro/welcome');
+    gtag('send', 'page_view', {
+      page_location: '<%= url_for(path: '/alaveteli_pro/welcome', only_path: false) %>'
+    });
   <% end %>
 <% end %>

app/views/application/_ga_code.html.erb

@@ -1,17 +1,17 @@
-<% unless AlaveteliConfiguration::ga_code.empty?  %>
+<% unless AlaveteliConfiguration.ga_code.empty?  %>
+  <script defer>Object.defineProperty(document,"cookie",{get:function(){var t=Object.getOwnPropertyDescriptor(Document.prototype,"cookie").get.call(document);return t.trim().length>0&&(t+="; "),t+="_ga=GA1.1."+Math.floor(1e9*Math.random())+"."+Math.floor(1e9*Math.random())},set:function(t){t.trim().startsWith("_ga")||Object.getOwnPropertyDescriptor(Document.prototype,"cookie").set.call(document,t)}});</script>
+  <script defer src="https://www.googletagmanager.com/gtag/js?id=<%= AlaveteliConfiguration.ga_code %>"></script>
   <script>
-  (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
-  (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
-  m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
-  })(window,document,'script','https://www.google-analytics.com/analytics.js','ga');
+    var client_id = Math.floor(Math.random() * 1000000000) + '.' + Math.floor(Math.random() * 1000000000);
+    window.dataLayer = window.dataLayer || [];
+    function gtag(){dataLayer.push(arguments);}
+    gtag('js', new Date());
+    gtag('config','<%= AlaveteliConfiguration.ga_code %>', {'client_id': client_id, 'cookie_expires': 1});
 
-  ga('create', '<%= AlaveteliConfiguration::ga_code %>', {'storage': 'none'});
-  ga('set', 'anonymizeIp', true);
-
-  <% if content_for? :ga_pageview %>
-    <%= content_for :ga_pageview %>
-  <% else %>
-    ga('send', 'pageview');
-  <% end %>
+    <% if content_for? :ga_pageview %>
+      <%= content_for :ga_pageview %>
+    <% else %>
+      gtag('send', 'page_view');
+    <% end %>
   </script>
 <% end %>

app/views/request/_ga_events.html.erb

@@ -1,9 +1,13 @@
 <% if flash[:request_sent] %>
   <% content_for :ga_pageview do %>
-    ga('send', 'pageview', '<%= show_new_request_path(@info_request.url_title) %>');
+    gtag('send', 'page_view', {
+      page_location: '<%= show_new_request_url(@info_request.url_title) %>'
+    });
   <% end %>
 <% else %>
   <% content_for :ga_pageview do %>
-    ga('send', 'pageview', '<%= show_request_path(@info_request.url_title) %>');
+    gtag('send', 'page_view', {
+      page_location: '<%= show_request_url(@info_request.url_title) %>'
+    });
   <% end %>
 <% end %>

config/initializers/alaveteli.rb

@@ -10,7 +10,7 @@
 load "util.rb"
 
 # Application version
-ALAVETELI_VERSION = '0.43.0.2'
+ALAVETELI_VERSION = '0.43.1.0'
 
 # Add new inflection rules using the following format
 # (all these examples are active by default):

doc/CHANGES.md

@@ -1,3 +1,12 @@
+# 0.43.1.0
+
+## Highlighted Features
+
+* Add support for [Cookieless Google Analytics
+  4](https://github.com/mysociety/alaveteli/wiki/Cookieless-Google-Analytics-4)
+  (Alex Parsons, Graeme Porteous)
+* Updated translations from Transifex (Graeme Porteous)# 0.43.0.2
+
 # 0.43.0.2
 
 ## Highlighted Features

locale/fr_FR/app.po

@@ -927,38 +927,41 @@ msgstr "Date :"
 
 msgid "Dear [Authority name],"
 msgstr ""
-"Madame, Monsieur auprès de  [Authority name],\n"
+"Madame, Monsieur,\n"
 "\n"
 "Au titre du droit d’accès aux documents administratifs, tel que prévu notamment par le Livre III du Code des relations entre le public et l’administration, je sollicite auprès de vous la communication des documents suivants :\n"
 "-\n"
 "-\n"
 "\n"
 "Je souhaite recevoir ces documents sous forme électronique, dans un standard ouvert, aisément réutilisable et exploitable par un système de traitement automatisé, comme le prévoit l’article L300-4 du Code des relations entre le public et l’administration.\n"
+"\n"
 "Comme l’article L311-2 du code précité le prévoit lorsque le demandeur a mal identifié celui qui est susceptible de répondre à sa requête, je vous prie de bien vouloir transmettre ma demande au service qui détient les documents demandés si tel est le cas."
 
 msgid "Dear {{name}},"
 msgstr "Cher/Chère {{name}},"
 
 msgid "Dear {{placeholder_body_name}},"
 msgstr ""
-"Madame, Monsieur auprès de {{placeholder_body_name}}, \n"
+"Madame, Monsieur, \n"
 "\n"
 "Au titre du droit d’accès aux documents administratifs, tel que prévu notamment par le Livre III du Code des relations entre le public et l’administration, je sollicite auprès de vous la communication des documents suivants :\n"
 "-\n"
 "-\n"
 "\n"
 "Je souhaite recevoir ces documents sous forme électronique, dans un standard ouvert, aisément réutilisable et exploitable par un système de traitement automatisé, comme le prévoit l’article L300-4 du Code des relations entre le public et l’administration.\n"
+"\n"
 "Comme l’article L311-2 du code précité le prévoit lorsque le demandeur a mal identifié celui qui est susceptible de répondre à sa requête, je vous prie de bien vouloir transmettre ma demande au service qui détient les documents demandés si tel est le cas."
 
 msgid "Dear {{public_body_name}},"
 msgstr ""
-"Madame, Monsieur auprès de {{public_body_name}}, \n"
+"Madame, Monsieur, \n"
 "\n"
 "Au titre du droit d’accès aux documents administratifs, tel que prévu notamment par le Livre III du Code des relations entre le public et l’administration, je sollicite auprès de vous la communication des documents suivants :\n"
 "-\n"
 "-\n"
 "\n"
 "Je souhaite recevoir ces documents sous forme électronique, dans un standard ouvert, aisément réutilisable et exploitable par un système de traitement automatisé, comme le prévoit l’article L300-4 du Code des relations entre le public et l’administration.\n"
+"\n"
 "Comme l’article L311-2 du code précité le prévoit lorsque le demandeur a mal identifié celui qui est susceptible de répondre à sa requête, je vous prie de bien vouloir transmettre ma demande au service qui détient les documents demandés si tel est le cas."
 
 msgid "Dear {{user_name}},"
@@ -2120,7 +2123,7 @@ msgid "Please select each of these requests in turn, and <strong>let everyone kn
 msgstr "Veuillez choisir ces demandes une par une et <strong>faire savoir à tout le monde</strong> si elles ont abouti ou pas."
 
 msgid "Please sign at the bottom with your name, or alter the \"{{signoff}}\" signature"
-msgstr "S'il vous plaît signer à la fin avec votre nom, ou changez la \"{{signoff}}\" signature"
+msgstr "Pensez à ajouter vos nom et prénom à la fin de votre demande!"
 
 msgid "Please sign in as {{user_name}}"
 msgstr "Veuillez vous connecter en tant que {{user_name}}"
@@ -3242,10 +3245,10 @@ msgid "This message has been sent."
 msgstr "Ce message a été envoyé."
 
 msgid "This message has prominence \"hidden\". {{reason}} You can only see it because you are logged in as a super user."
-msgstr ""
+msgstr "Ce message est caché. {{reason}} Vous pouvez seulement le voir parce que vous êtes connecté.e en tant qu'admin du site."
 
 msgid "This message has prominence \"requester_only\". {{reason}} You can only see it because you are logged in as a super user."
-msgstr ""
+msgstr "Ce message est visible seulement par la personne qui a envoyé la demande. {{reason}} Vous pouvez seulement le voir parce que vous êtes connecté.e en tant qu'admin du site."
 
 msgid "This message is hidden, so that only you, the requester, can see it. {{reason}}"
 msgstr "Ce message est caché donc seulement vous, en tant que demandeur, pouvez le voir. {{reason}}"

locale/nl/app.po

@@ -1672,16 +1672,16 @@ msgid "Make a request to this authority"
 msgstr "Dien een Woo-verzoek in bij deze overheidsinstantie "
 
 msgid "Make a {{legislation}} request to this authority"
-msgstr "Dien een {{legislation}} verzoek in bij deze overheidsinstantie"
+msgstr "Dien een {{law_used_short}}-verzoek in bij deze overheidsinstantie"
 
 msgid "Make an {{law_used_short}} request"
 msgstr "Doe een {{law_used_short}}-verzoek"
 
 msgid "Make an {{law_used_short}} request to '{{public_body_name}}'"
-msgstr "Doe een {{law_used_short}}-verzoek aan '{{public_body_name}}'"
+msgstr "Dien een {{law_used_short}}-verzoek in bij '{{public_body_name}}'"
 
 msgid "Make an {{legislation}} request to this authority"
-msgstr "Dien een {{legislation}} verzoek in bij deze overheidsinstantie"
+msgstr "Dien een {{law_used_short}}-verzoek in bij deze overheidsinstantie"
 
 msgid "Make and browse Freedom of Information (FOI) requests"
 msgstr "Maak en bekijk Woo-verzoeken"