README.md

@@ -97,7 +97,7 @@ You must publish a port on your host machine to bind to the container's web port
 An example of a minimum run configuration to access maloja via `localhost:42010`:
 
 ```console
-	docker run -p 42010:42010 -v $PWD/malojadata:/mljdata -e MALOJA_DATA_DIRECTORY=/mljdata krateng/maloja
+	docker run -p 42010:42010 -v $PWD/malojadata:/mljdata -e MALOJA_DATA_DIRECTORY=/mljdata maloja
 ```
 
 ### Extras

dev/releases/3.0.yml

@@ -28,11 +28,3 @@ minor_release_name: "Yeonhee"
     - "[Feature] Added better feedback to native API endpoints"
     - "[Bugfix] Fixed native API receiving superfluous keywords"
     - "[Bugfix] Fixed crash when importing scrobbles with artists with similar names"
-3.0.5:
-    commit: "fe21894c5ecf3a53c9c5c00453abfc7f41c6a83e"
-    notes:
-    - "[Feature] Added notification system for web interface"
-    - "[Bugfix] Fixed crash when encountering error in Lastfm import"
-3.0.6:
-    notes:
-    - "[Bugfix] Better parsing of featuring artists"

dev/releases/branch.yml

@@ -1,3 +1,2 @@
-- "[Architecture] Cleaned up legacy process control"
 - "[Feature] Implemented track title and artist name editing from web interface"
 - "[Feature] Implemented track and artist merging from web interface"

maloja/__pkginfo__.py

@@ -4,7 +4,7 @@
 # you know what f*ck it
 # this is hardcoded for now because of that damn project / package name discrepancy
 # i'll fix it one day
-VERSION = "3.0.5"
+VERSION = "3.0.4"
 HOMEPAGE = "https://github.com/krateng/maloja"
 
 

maloja/cleanup.py

@@ -109,9 +109,9 @@ class CleanerAgent:
 
 
 		for d in self.delimiters_feat:
-			if re.match(r"(.*) [\(\[]" + d + " (.*)[\)\]]",a) is not None:
-				return self.parseArtists(re.sub(r"(.*) [\(\[]" + d + " (.*)[\)\]]",r"\1",a)) + \
-						self.parseArtists(re.sub(r"(.*) [\(\[]" + d + " (.*)[\)\]]",r"\2",a))
+			if re.match(r"(.*) \(" + d + " (.*)\)",a) is not None:
+				return self.parseArtists(re.sub(r"(.*) \(" + d + " (.*)\)",r"\1",a)) + \
+						self.parseArtists(re.sub(r"(.*) \(" + d + " (.*)\)",r"\2",a))
 
 
 
@@ -141,11 +141,9 @@ class CleanerAgent:
 
 		t = t.replace("[","(").replace("]",")")
 
-		# we'll leave these matching all bracket types so future changes
-		# won't require readaption
-		t = re.sub(r" [\(\[]as made famous by .*?[\)\]]","",t)
-		t = re.sub(r" [\(\[]originally by .*?[\)\]]","",t)
-		t = re.sub(r" [\(\[].*?Remaster.*?[\)\]]","",t)
+		t = re.sub(r" \(as made famous by .*?\)","",t)
+		t = re.sub(r" \(originally by .*?\)","",t)
+		t = re.sub(r" \(.*?Remaster.*?\)","",t)
 
 		for s in malojaconfig["REMOVE_FROM_TITLE"]:
 			if s in t:
@@ -158,9 +156,9 @@ class CleanerAgent:
 
 	def parseTitleForArtists(self,t):
 		for d in self.delimiters_feat:
-			if re.match(r"(.*) [\(\[]" + d + " (.*?)[\)\]]",t) is not None:
-				(title,artists) = self.parseTitleForArtists(re.sub(r"(.*) [\(\[]" + d + " (.*?)[\)\]]",r"\1",t))
-				artists += self.parseArtists(re.sub(r"(.*) [\(\[]" + d + " (.*?)[\)\]].*",r"\2",t))
+			if re.match(r"(.*) \(" + d + " (.*?)\)",t) is not None:
+				(title,artists) = self.parseTitleForArtists(re.sub(r"(.*) \(" + d + " (.*?)\)",r"\1",t))
+				artists += self.parseArtists(re.sub(r"(.*) \(" + d + " (.*?)\).*",r"\2",t))
 				return (title,artists)
 			if re.match(r"(.*) - " + d + " (.*)",t) is not None:
 				(title,artists) = self.parseTitleForArtists(re.sub(r"(.*) - " + d + " (.*)",r"\1",t))

maloja/proccontrol/tasks/import_scrobbles.py

@@ -268,17 +268,11 @@ def parse_lastfm(inputf):
 	with open(inputf,'r',newline='') as inputfd:
 		reader = csv.reader(inputfd)
 
-		line = 0
 		for row in reader:
-			line += 1
 			try:
 				artist,album,title,time = row
 			except ValueError:
-				yield ('FAIL',None,f"{row} (Line {line}) does not look like a valid entry. Scrobble not imported.")
-				continue
-
-			if time == '':
-				yield ('FAIL',None,f"{row} (Line {line}) is missing a timestamp.")
+				yield ('FAIL',None,f"{row} does not look like a valid entry. Scrobble not imported.")
 				continue
 
 			try:
@@ -293,7 +287,7 @@ def parse_lastfm(inputf):
 					'scrobble_duration':None
 				},'')
 			except Exception as e:
-				yield ('FAIL',None,f"{row} (Line {line}) could not be parsed. Scrobble not imported. ({repr(e)})")
+				yield ('FAIL',None,f"{entry} could not be parsed. Scrobble not imported. ({repr(e)})")
 				continue
 
 

maloja/web/jinja/abstracts/base.jinja

@@ -50,7 +50,9 @@
 			{% endblock %}
 		{% endblock %}
 
+		<div id="notification_area">
 
+		</div>
 
 
 		<div class="footer">
@@ -87,10 +89,6 @@
 			{% include 'icons/settings.jinja' %}
 		</div>
 
-		<div id="notification_area">
-
-		</div>
-
 
 	</body>
 </html>

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "malojaserver"
-version = "3.0.5"
+version = "3.0.4"
 description = "Self-hosted music scrobble database"
 readme = "./README.md"
 requires-python = ">=3.6"