diff --git a/apps/i2psnark/java/src/org/klomp/snark/PeerCoordinator.java b/apps/i2psnark/java/src/org/klomp/snark/PeerCoordinator.java
index 9e914c4194..285af4d19d 100644
--- a/apps/i2psnark/java/src/org/klomp/snark/PeerCoordinator.java
+++ b/apps/i2psnark/java/src/org/klomp/snark/PeerCoordinator.java
@@ -1524,7 +1524,9 @@ class PeerCoordinator implements PeerListener
* @since 0.9.31
*/
public void gotCommentReq(Peer peer, int num) {
- /* if disabled, return */
+ /* TODO cache per-torrent setting, use it instead */
+ if (!_util.utCommentsEnabled())
+ return;
CommentSet comments = snark.getComments();
if (comments != null) {
int lastSent = peer.getTotalCommentsSent();
@@ -1547,7 +1549,9 @@ class PeerCoordinator implements PeerListener
* @since 0.9.31
*/
public void gotComments(Peer peer, List comments) {
- /* if disabled, return */
+ /* TODO cache per-torrent setting, use it instead */
+ if (!_util.utCommentsEnabled())
+ return;
if (!comments.isEmpty())
snark.addComments(comments);
}
diff --git a/apps/i2psnark/java/src/org/klomp/snark/SnarkManager.java b/apps/i2psnark/java/src/org/klomp/snark/SnarkManager.java
index 09b92d53d8..061ccd727f 100644
--- a/apps/i2psnark/java/src/org/klomp/snark/SnarkManager.java
+++ b/apps/i2psnark/java/src/org/klomp/snark/SnarkManager.java
@@ -797,6 +797,9 @@ public class SnarkManager implements CompleteListener, ClientApp {
updateConfig();
}
+ /**
+ * @since 0.9.31
+ */
public boolean getUniversalTheming() {
return _context.getBooleanProperty(RC_PROP_UNIVERSAL_THEMING);
}
diff --git a/apps/i2psnark/java/src/org/klomp/snark/comments/Comment.java b/apps/i2psnark/java/src/org/klomp/snark/comments/Comment.java
index 8ac662efb6..86f9eee747 100644
--- a/apps/i2psnark/java/src/org/klomp/snark/comments/Comment.java
+++ b/apps/i2psnark/java/src/org/klomp/snark/comments/Comment.java
@@ -70,7 +70,7 @@ public class Comment implements Comparable {
name = name.substring(0, MAX_NAME_LEN);
}
this.name = name;
- if (rating < 0 || rating > 5)
+ if (rating < 0)
rating = 0;
else if (rating > 5)
rating = 5;
diff --git a/apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPClient.java b/apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPClient.java
index f4f74cf040..e4c61fed36 100644
--- a/apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPClient.java
+++ b/apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPClient.java
@@ -1375,8 +1375,10 @@ public class I2PTunnelHTTPClient extends I2PTunnelHTTPClientBase implements Runn
if(_context.namingService().getName().equals("BlockfileNamingService")) {
out.write(" " + _t("If you want to keep track of sites you have added manually, add to your Master or Private address book instead."));
}
+ // FIXME wasn't escaped
+ String label = _t("Save & continue").replace("&", "&");
out.write("
\n\n");
+ label + "\n");
if(_context.namingService().getName().equals("BlockfileNamingService")) {
// only blockfile supports multiple books
@@ -1384,12 +1386,12 @@ public class I2PTunnelHTTPClient extends I2PTunnelHTTPClientBase implements Runn
out.write("
" + _t("Save {0} to master address book and continue to website", destination) + "
\n
" +
_t("This address will be saved to your Master address book. Select this option for addresses you wish to keep separate from the main router address book, but don't mind publishing.") +
"
\n\n");
+ label + "\n");
out.write("
" + _t("Save {0} to private address book and continue to website", destination) + "
\n
" +
_t("This address will be saved to your Private address book, ensuring it is never published.") +
"
\n\n");
+ label + "\n");
}
// Firefox (and others?) don't send referer to meta refresh target, which is
diff --git a/apps/routerconsole/java/src/net/i2p/router/web/HomeHelper.java b/apps/routerconsole/java/src/net/i2p/router/web/HomeHelper.java
index 41bd52a8db..4932f3f88a 100644
--- a/apps/routerconsole/java/src/net/i2p/router/web/HomeHelper.java
+++ b/apps/routerconsole/java/src/net/i2p/router/web/HomeHelper.java
@@ -29,7 +29,8 @@ public class HomeHelper extends HelperBase {
static final String DEFAULT_SERVICES =
_x("Addressbook") + S + _x("Manage your I2P hosts file here (I2P domain name resolution)") + S + "/dns" + S + I + "book_addresses.png" + S +
_x("Configure Bandwidth") + S + _x("I2P Bandwidth Configuration") + S + "/config" + S + I + "info/bandwidth.png" + S +
- _x("Configure UI") + S + _x("Select console theme & language & set optional console password") + S + "/configui" + S + I + "info/ui.png" + S +
+ // FIXME wasn't escaped
+ _x("Configure UI") + S + _x("Select console theme & language & set optional console password").replace("&", "&") + S + "/configui" + S + I + "info/ui.png" + S +
_x("Customize Home Page") + S + _x("I2P Home Page Configuration") + S + "/confighome" + S + I + "home_page.png" + S +
_x("Email") + S + _x("Anonymous webmail client") + S + "/susimail/susimail" + S + I + "email.png" + S +
_x("Help") + S + _x("I2P Router Help") + S + "/help" + S + I + "support.png" + S +
@@ -229,7 +230,7 @@ public class HomeHelper extends HelperBase {
.append(app.name)
.append("\">")
.append(DataHelper.escapeHTML(app.name))
- .append("