Add optional parameter to texture querying methods

And remove the parameterised flavors to avoid code duplication.

Sure, it's a bit more code now, but having just one SQL query for
each case is less error-prone.

app/src/main/java/de/markusfisch/android/shadereditor/database/Database.java

@@ -152,7 +152,8 @@ public Cursor getShaders() {
 				null);
 	}
 
-	public Cursor getTextures() {
+	public Cursor getTextures(String substring) {
+		boolean useSubstring = substring != null;
 		return db.rawQuery(
 				"SELECT " +
 						TEXTURES_ID + "," +
@@ -162,25 +163,17 @@ public Cursor getTextures() {
 						TEXTURES_THUMB +
 						" FROM " + TEXTURES +
 						" WHERE " + TEXTURES_RATIO + " = 1" +
+						(useSubstring
+								? " AND " + TEXTURES_NAME + " LIKE ?"
+								: "") +
 						" ORDER BY " + TEXTURES_ID,
-				null);
-	}
-
-	public Cursor getTexturesWith(String substring) {
-		return db.rawQuery(
-				"SELECT " +
-						TEXTURES_ID + "," +
-						TEXTURES_NAME + "," +
-						TEXTURES_WIDTH + "," +
-						TEXTURES_HEIGHT + "," +
-						TEXTURES_THUMB +
-						" FROM " + TEXTURES +
-						" WHERE " + TEXTURES_RATIO + " = 1 AND " + TEXTURES_NAME + " LIKE ?" +
-						" ORDER BY " + TEXTURES_ID,
-				new String[]{"%" + substring + "%"});
+				useSubstring
+						? new String[]{"%" + substring + "%"}
+						: null);
 	}
 
-	public Cursor getSamplerCubeTextures() {
+	public Cursor getSamplerCubeTextures(String substring) {
+		boolean useSubstring = substring != null;
 		return db.rawQuery(
 				"SELECT " +
 						TEXTURES_ID + "," +
@@ -190,22 +183,13 @@ public Cursor getSamplerCubeTextures() {
 						TEXTURES_THUMB +
 						" FROM " + TEXTURES +
 						" WHERE " + TEXTURES_RATIO + " = 1.5" +
+						(useSubstring
+								? " AND " + TEXTURES_NAME + " LIKE ?"
+								: "") +
 						" ORDER BY " + TEXTURES_ID,
-				null);
-	}
-
-	public Cursor getSamplerCubeTexturesWith(String substring) {
-		return db.rawQuery(
-				"SELECT " +
-						TEXTURES_ID + "," +
-						TEXTURES_NAME + "," +
-						TEXTURES_WIDTH + "," +
-						TEXTURES_HEIGHT + "," +
-						TEXTURES_THUMB +
-						" FROM " + TEXTURES +
-						" WHERE " + TEXTURES_RATIO + " = 1.5 AND " + TEXTURES_NAME + " LIKE ?" +
-						" ORDER BY " + TEXTURES_ID,
-				new String[]{"%" + substring + "%"});
+				useSubstring
+						? new String[]{"%" + substring + "%"}
+						: null);
 	}
 
 	public boolean isShaderAvailable(long id) {

app/src/main/java/de/markusfisch/android/shadereditor/fragment/UniformSampler2dPageFragment.java

@@ -101,11 +101,7 @@ protected void addTexture() {
 	}
 
 	protected Cursor loadTextures() {
-		if (searchQuery == null) {
-			return ShaderEditorApp.db.getTextures();
-		}
-
-		return ShaderEditorApp.db.getTexturesWith(searchQuery);
+		return ShaderEditorApp.db.getTextures(searchQuery);
 	}
 
 	private void showTexture(long id) {

app/src/main/java/de/markusfisch/android/shadereditor/fragment/UniformSamplerCubePageFragment.java

@@ -27,10 +27,6 @@ protected void addTexture() {
 
 	@Override
 	protected Cursor loadTextures() {
-		if (searchQuery == null) {
-			return ShaderEditorApp.db.getSamplerCubeTextures();
-		}
-
-		return ShaderEditorApp.db.getSamplerCubeTexturesWith(searchQuery);
+		return ShaderEditorApp.db.getSamplerCubeTextures(searchQuery);
 	}
 }

app/src/main/java/de/markusfisch/android/shadereditor/widget/BackBufferParametersView.java

@@ -48,7 +48,7 @@ protected void onAttachedToWindow() {
 
 		MergeCursor mergeCursor = new MergeCursor(new Cursor[]{
 				matrixCursor,
-				ShaderEditorApp.db.getTextures()
+				ShaderEditorApp.db.getTextures(null)
 		});
 
 		adapter = new TextureSpinnerAdapter(context, mergeCursor);