Karl/BobStore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'trim-names-and-summaries' into 'master'

Browse Source 
Trim names and summaries

See merge request fdroid/fdroidclient!613
This commit is contained in:
Hans-Christoph Steiner 2017-12-01 11:17:12 +00:00
commit 92f94ef195
2 changed files with 20 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/src/main/java/org/fdroid/fdroid/data/AppProvider.java
View File

@ -944,6 +944,10 @@ public class AppProvider extends FDroidProvider {
values.put(Cols.DESCRIPTION, "");
}
// Trim these to avoid unwanted newlines in the UI
values.put(Cols.SUMMARY, values.getAsString(Cols.SUMMARY).trim());
values.put(Cols.NAME, values.getAsString(Cols.NAME).trim());
String[] categories = null;
boolean saveCategories = false;
if (values.containsKey(Cols.ForWriting.Categories.CATEGORIES)) {

16
app/src/test/java/org/fdroid/fdroid/data/AppProviderTest.java
View File

@ -267,6 +267,16 @@ public class AppProviderTest extends FDroidProviderTest {
assertEquals("F-Droid", otherApp.name);
}
@Test
public void testInsertTrimsNamesAndSummary() {
// Insert a new record with unwanted newlines...
App app = insertApp("org.fdroid.trimmer", "Trim me\n", "Trim me too\n");
assertEquals("org.fdroid.trimmer", app.packageName);
assertEquals("Trim me", app.name);
assertEquals("Trim me too", app.summary);
}
/**
* We intentionally throw an IllegalArgumentException if you haven't
* yet called cursor.move*().
@ -296,6 +306,12 @@ public class AppProviderTest extends FDroidProviderTest {
insertApp(contentResolver, context, id, name, new ContentValues());
}
private App insertApp(String id, String name, String summary) {
ContentValues additionalValues = new ContentValues();
additionalValues.put(Cols.SUMMARY, summary);
return insertApp(contentResolver, context, id, name, additionalValues);
}
public static App insertApp(ShadowContentResolver contentResolver, Context context, String id, String name, ContentValues additionalValues) {
return insertApp(contentResolver, context, id, name, additionalValues, 1);
}