diff main/NetworkPermissionTester.cpp @ 1613:3f6c18fcc075

Rework network permission tester to note that aggregate info may be used, and tweak for clarity
author Chris Cannam
date Sat, 25 Feb 2017 19:24:13 +0000
parents 7f7fba33b7de
children 3dbfe3ba0d1f
line wrap: on
line diff
--- a/main/NetworkPermissionTester.cpp	Fri Feb 24 15:57:13 2017 +0000
+++ b/main/NetworkPermissionTester.cpp	Sat Feb 25 19:24:13 2017 +0000
@@ -46,24 +46,43 @@
 	QGridLayout *layout = new QGridLayout;
 	d.setLayout(layout);
 
+        QString preamble;
+        preamble = QCoreApplication::translate
+            ("NetworkPermissionTester",
+             "<h2>Welcome to Sonic Visualiser!</h2>"
+             "<p><img src=\":icons/qm-logo-smaller.png\" style=\"float:right\">Sonic Visualiser is a program for viewing and exploring audio data for semantic music analysis and annotation.</p>"
+             "<p>Developed in the Centre for Digital Music at Queen Mary University of London, Sonic Visualiser is open source software under the GNU General Public License.</p>"
+             "<p><hr></p>"
+             "<p><b>Before we go on...</b></p>"
+             "<p>Sonic Visualiser would like permission to use the network.</p>");
+
+        QString bullets;
+        if (m_withOSC) {
+            bullets = QCoreApplication::translate
+                ("NetworkPermissionTester",
+                 "<p>This is to:</p>"
+                 "<ul><li> Find information about available and installed plugins;</li>"
+                 "<li> Support the use of Open Sound Control; and</li>"
+                 "<li> Tell you when updates are available.</li>"
+                 "</ul>");
+        } else {
+            bullets = QCoreApplication::translate
+                ("NetworkPermissionTester",
+                 "<p>This is to:</p>"
+                 "<ul><li> Find information about available and installed plugins; and</li>"
+                 "<li> Tell you when updates are available.</li>"
+                 "</ul>");
+        }
+
+        QString postamble;
+        postamble = QCoreApplication::translate
+            ("NetworkPermissionTester",
+             "<p><b>No personal information will be sent, no tracking is carried out, and no individual information will be shared with anyone else.</b> We will however make aggregate counts of distinct requests for usage reporting.</p>"
+             "<p>We recommend that you allow this, because it makes Sonic Visualiser more useful to you and supports the public funding of this work. But if you do not wish to allow it, please un-check the box below.<br></p>");
+        
 	QLabel *label = new QLabel;
 	label->setWordWrap(true);
-	label->setText
-	    (QCoreApplication::translate
-	     ("NetworkPermissionTester",
-	      "<h2>Welcome to Sonic Visualiser!</h2>"
-	      "<p><img src=\":icons/qm-logo-smaller.png\" style=\"float:right\">Sonic Visualiser is a program for viewing and exploring audio data for semantic music analysis and annotation.</p>"
-	      "<p>Developed in the Centre for Digital Music at Queen Mary, University of London, Sonic Visualiser is provided free as open source software under the GNU General Public License.</p>"
-              "<p><hr></p>"
-	      "<p><b>Before we go on...</b></p>"
-	      "<p>Sonic Visualiser would like to make networking connections and open a network port.</p>"
-	      "<p>This is to:</p>"
-	      "<ul><li> Find information about available and installed plugins;</li>"
-	      "<li> Support the use of Open Sound Control, where configured; and</li>"
-	      "<li> Tell you when updates are available.</li>"
-              "</ul>"
-	      "<p>No personal information will be sent, no tracking is carried out, and all requests happen in the background without interrupting your work.</p>"
-	      "<p>We recommend that you allow this, because it makes Sonic Visualiser more useful. But if you do not wish to do so, please un-check the box below.<br></p>"));
+	label->setText(preamble + bullets + postamble);
 	layout->addWidget(label, 0, 0);
 
 	QCheckBox *cb = new QCheckBox(QCoreApplication::translate("NetworkPermissionTester", "Allow this"));