Automatically enable acceptable ads and notify the user about it

src/org/adblockplus/android/Preferences.java

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2013 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 139 matching lines @@
 
     boolean firstRun = firstRunActionsPending && application.isFirstRun();
     firstRunActionsPending = false;
 
     if (firstRun && current != null)
     {
       showNotificationDialog(getString(R.string.install_name),
           String.format(getString(R.string.msg_subscription_offer, current.title)),
           application.getAcceptableAdsUrl());
       application.setNotifiedAboutAcceptableAds(true);
-      application.setAcceptableAdsEnabled(true);  
+      application.setAcceptableAdsEnabled(true);
     }
     else if (!application.isNotifiedAboutAcceptableAds())

[Wladimir Palant, 2013/11/26 11:01:02]

Shouldn't you check whether acceptable ads are already enabled, just to be safe?

[Felix Dahlke, 2013/11/26 13:59:31]

Why? Even if they were, we want to notify every user at least once about them.

     {
       showNotificationDialog(getString(R.string.acceptableads_name),
           getString(R.string.msg_acceptable_ads), application.getAcceptableAdsUrl());

[Wladimir Palant, 2013/11/26 11:01:02]

Style nit: add a line break to put the last parameter on a separate line?

[Felix Dahlke, 2013/11/26 13:59:31]

I'm wrapping Java code to 100 columns currently, as suggested by the Mozilla
Java style guide.

       application.setNotifiedAboutAcceptableAds(true);
       application.setAcceptableAdsEnabled(true);
     }
 
     // Enable manual subscription refresh
     subscriptionList.setOnRefreshClickListener(new View.OnClickListener()
     {
       @Override
       public void onClick(View v)
       {
@@ skipping 28 matching lines @@
     if (enabled || firstRun)
       setFilteringEnabled(true);
     if (enabled || firstRun || (proxyenabled && !autoconfigured))
       setProxyEnabled(true);
 
     bindService(new Intent(this, ProxyService.class), proxyServiceConnection, 0);
   }
 
   private void showNotificationDialog(String title, String message, String url)
   {
+    url = TextUtils.htmlEncode(url);
     message = TextUtils.htmlEncode(message)
         .replaceAll("&lt;a&gt;(.*?)&lt;/a&gt;", "<a href=\"" + url + "\">$1</a>");

[Wladimir Palant, 2013/11/26 11:01:02]

Comment from previous review (encode entities in URL) applies here as well.

[Felix Dahlke, 2013/11/26 13:59:31]

Done.

     final TextView messageView = new TextView(this);
     messageView.setText(Html.fromHtml(message));
     messageView.setMovementMethod(LinkMovementMethod.getInstance());
     final int padding = 10;
     messageView.setPadding(padding, padding, padding, padding);
     new AlertDialog.Builder(this).setTitle(title)
         .setView(messageView)
         .setIcon(android.R.drawable.ic_dialog_info)
         .setPositiveButton(R.string.ok, null).create().show();
   }
@@ skipping 334 matching lines @@
         showConfigurationMsg(getString(R.string.msg_configuration));
     }
 
     public void onServiceDisconnected(ComponentName className)
     {
       proxyService = null;
       Log.d(TAG, "Proxy service disconnected");
     }
   };
 }