Skip to content

Commit fa61b86

Browse files
committed
Code improvements
1 parent 98268e3 commit fa61b86

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

extractor/src/main/java/org/schabi/newpipe/extractor/services/bandcamp/extractors/BandcampExtractorHelper.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,15 +43,15 @@ public static JsonObject getJsonData(final String html, final String variable)
4343

4444
/**
4545
* Translate all these parameters together to the URL of the corresponding album or track
46-
* using the mobile api
46+
* using the mobile API
4747
*/
4848
public static String getStreamUrlFromIds(final long bandId, final long itemId, final String itemType)
4949
throws ParsingException {
5050

5151
try {
5252
final String jsonString = NewPipe.getDownloader().get(
5353
"https://bandcamp.com/api/mobile/22/tralbum_details?band_id=" + bandId
54-
+ "&tralbum_id=" + itemId + "&tralbum_type=" + itemType.substring(0, 1))
54+
+ "&tralbum_id=" + itemId + "&tralbum_type=" + itemType.charAt(0))
5555
.responseBody();
5656

5757
return JsonParser.object().from(jsonString).getString("bandcamp_url").replace("http://", "https://");
@@ -88,7 +88,7 @@ public static String smartConcatenate(final String[] strings, final String comma
8888

8989
}
9090

91-
return String.valueOf(result);
91+
return result.toString();
9292
}
9393

9494
/**

extractor/src/main/java/org/schabi/newpipe/extractor/services/bandcamp/extractors/BandcampRadioInfoItemExtractor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414

1515
public class BandcampRadioInfoItemExtractor implements StreamInfoItemExtractor {
1616

17-
private JsonObject show;
17+
private final JsonObject show;
1818

1919
public BandcampRadioInfoItemExtractor(final JsonObject radioShow) {
2020
show = radioShow;

extractor/src/main/java/org/schabi/newpipe/extractor/services/bandcamp/extractors/BandcampSuggestionExtractor.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public List<String> suggestionList(final String query) throws IOException, Extra
3737
final JsonArray jsonArray = fuzzyResults.getObject("auto")
3838
.getArray("results");
3939

40-
final ArrayList<String> suggestions = new ArrayList<>();
40+
final List<String> suggestions = new ArrayList<>();
4141

4242
for (final Object fuzzyResult : jsonArray) {
4343
final String res = ((JsonObject) fuzzyResult).getString("name");
@@ -52,6 +52,5 @@ public List<String> suggestionList(final String query) throws IOException, Extra
5252
return Collections.emptyList();
5353
}
5454

55-
5655
}
5756
}

extractor/src/test/java/org/schabi/newpipe/extractor/services/bandcamp/BandcampStreamLinkHandlerFactoryTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,5 +48,6 @@ public void testAcceptUrl() throws ParsingException {
4848
assertTrue(linkHandler.acceptUrl("http://ZachBenson.Bandcamp.COM/Track/U-I-Tonite/"));
4949
assertTrue(linkHandler.acceptUrl("https://interovgm.com/track/title"));
5050
assertTrue(linkHandler.acceptUrl("http://bandcamP.com/?show=38"));
51+
assertTrue(linkHandler.acceptUrl("https://goodgoodblood-tl.bandcamp.com/track/when-it-all-wakes-up"));
5152
}
5253
}

0 commit comments

Comments
 (0)