From cf300a2a912433c122df7ff745075cd888c1593b Mon Sep 17 00:00:00 2001
From: Hellow <Lars.Noack@outlook.de>
Date: Sat, 14 Jan 2023 14:41:40 +0100
Subject: [PATCH] fuck u

---
 src/goof.py                                   |   6 ++--
 src/music_kraken/__init__.py                  |   1 +
 src/music_kraken/database/__init__.py         |   1 +
 src/music_kraken/database/objects/__init__.py |   1 +
 src/music_kraken/database/objects/metadata.py |  32 ++++++++++++------
 src/music_kraken/database/objects/song.py     |  11 +++---
 src/test.db                                   | Bin 69632 -> 69632 bytes
 7 files changed, 34 insertions(+), 18 deletions(-)

diff --git a/src/goof.py b/src/goof.py
index 49de36c..d392223 100644
--- a/src/goof.py
+++ b/src/goof.py
@@ -5,7 +5,8 @@ from music_kraken import (
     Target,
     Source,
     Album,
-    Artist
+    Artist,
+    ID3Timestamp
 )
 
 from music_kraken.tagging import (
@@ -16,7 +17,6 @@ from music_kraken.tagging import (
 
 import music_kraken.database.new_database as db
 
-import datetime
 import pycountry
 
 
@@ -43,7 +43,7 @@ feature_artist = Artist(
 
 album_input = Album(
     title="One Final Action",
-    date=datetime.date(1986, 3, 1),
+    date=ID3Timestamp(year=1986, month=3, day=1),
     language=pycountry.languages.get(alpha_2="en"),
     label="cum productions"
 )
diff --git a/src/music_kraken/__init__.py b/src/music_kraken/__init__.py
index 735db13..7094e79 100644
--- a/src/music_kraken/__init__.py
+++ b/src/music_kraken/__init__.py
@@ -44,6 +44,7 @@ Source = database.Source
 Target = database.Target
 Lyrics = database.Lyrics
 Album = database.Album
+ID3Timestamp = database.ID3Timestamp
 MetadataSearch = metadata.MetadataSearch
 MetadataDownload = metadata.MetadataDownload
 
diff --git a/src/music_kraken/database/__init__.py b/src/music_kraken/database/__init__.py
index 8dd8fd4..07410fe 100644
--- a/src/music_kraken/database/__init__.py
+++ b/src/music_kraken/database/__init__.py
@@ -3,6 +3,7 @@ from . import (
     objects
 )
 
+ID3Timestamp = objects.ID3Timestamp
 Song = objects.Song
 Source = objects.Source
 Target = objects.Target
diff --git a/src/music_kraken/database/objects/__init__.py b/src/music_kraken/database/objects/__init__.py
index 0f77c78..a14ddb3 100644
--- a/src/music_kraken/database/objects/__init__.py
+++ b/src/music_kraken/database/objects/__init__.py
@@ -5,6 +5,7 @@ from . import (
 )
 
 ID3_MAPPING = metadata.Mapping
+ID3Timestamp = metadata.ID3Timestamp
 
 Song = song.Song
 Artist = song.Artist
diff --git a/src/music_kraken/database/objects/metadata.py b/src/music_kraken/database/objects/metadata.py
index 859f096..b764be2 100644
--- a/src/music_kraken/database/objects/metadata.py
+++ b/src/music_kraken/database/objects/metadata.py
@@ -1,5 +1,7 @@
 from enum import Enum
 from typing import List, Dict, Tuple
+
+import dateutil.tz
 from mutagen import id3
 import datetime
 
@@ -98,7 +100,7 @@ class Mapping(Enum):
             return cls.get_url_instance(key, value)
 
 
-class ID3Timestamp(datetime.datetime):
+class ID3Timestamp():
     def __init__(
             self,
             year: int = None,
@@ -109,7 +111,6 @@ class ID3Timestamp(datetime.datetime):
             second: int = None,
             microsecond=0,
             tzinfo=None,
-            *,
             fold=0
     ):
         self.has_year = year is not None
@@ -126,7 +127,10 @@ class ID3Timestamp(datetime.datetime):
             month = 1
         if not self.has_day:
             day = 1
-        super().__init__(
+
+        # https://stackoverflow.com/questions/399022/why-cant-i-subclass-datetime-date
+        super().__new__(
+            cls=type(self),
             year=year,
             month=month,
             day=day,
@@ -188,15 +192,8 @@ class Metadata:
     Shall only be read or edited via the Song object.
     call it like a dict to read/write values
     """
-    class FrameValue:
-        def __init__(self, values: list, modified_by: str) -> None:
-            """
-            Parameters:
-                values (list): the values.
-            """
-            pass
 
-    def __init__(self, data: dict = {}) -> None:
+    def __init__(self) -> None:
         # this is pretty self-explanatory
         # the key is a 4 letter key from the id3 standards like TITL
 
@@ -251,6 +248,16 @@ class Metadata:
 
         list_data = self.id3_attributes[key]
 
+        # convert for example the time objects to timestamps
+        for i, element in enumerate(list_data):
+            # for performance’s sake I don't do other checks if it is already the right type
+            if type(element) == str:
+                continue
+
+            if type(element) == ID3Timestamp:
+                list_data[i] = element.timestamp
+                continue
+
         """
         Version 2.4 of the specification prescribes that all text fields (the fields that start with a T, except for TXXX) can contain multiple values separated by a null character. 
         Thus if above conditions are met, I concatenate the list,
@@ -265,6 +272,9 @@ class Metadata:
         return Mapping.get_mutagen_instance(Mapping(key), self.get_id3_value(key))
 
     def __iter__(self):
+        # set the tagging timestamp to the current time
+        self.__setitem__(Mapping.TAGGING_TIME.value, [ID3Timestamp.now()])
+
         for key in self.id3_attributes:
             yield key, self.get_mutagen_object(key)
 
diff --git a/src/music_kraken/database/objects/song.py b/src/music_kraken/database/objects/song.py
index bb278f9..579a98c 100644
--- a/src/music_kraken/database/objects/song.py
+++ b/src/music_kraken/database/objects/song.py
@@ -5,7 +5,8 @@ import pycountry
 
 from .metadata import (
     Mapping as ID3_MAPPING,
-    Metadata
+    Metadata,
+    ID3Timestamp
 )
 from ...utils.shared import (
     MUSIC_DIR,
@@ -282,7 +283,7 @@ class Album(DatabaseObject, ID3Metadata):
             label: str = None,
             album_status: str = None,
             language: pycountry.Languages = None,
-            date: datetime.date = None,
+            date: ID3Timestamp = None,
             country: str = None,
             barcode: str = None,
             is_split: bool = False,
@@ -294,11 +295,12 @@ class Album(DatabaseObject, ID3Metadata):
         self.album_status: str = album_status
         self.label = label
         self.language: pycountry.Languages = language
-        self.date: datetime.date = date
+        self.date: ID3Timestamp = date
         self.country: str = country
         """
         TODO
         find out the id3 tag for barcode and implement it
+        maybee look at how mutagen does it with easy_id3
         """
         self.barcode: str = barcode
         self.is_split: bool = is_split
@@ -335,7 +337,8 @@ class Album(DatabaseObject, ID3Metadata):
             ID3_MAPPING.ALBUM: [self.title],
             ID3_MAPPING.COPYRIGHT: [self.copyright],
             ID3_MAPPING.LANGUAGE: [self.iso_639_2_language],
-            ID3_MAPPING.ALBUM_ARTIST: [a.name for a in self.artists]
+            ID3_MAPPING.ALBUM_ARTIST: [a.name for a in self.artists],
+            ID3_MAPPING.DATE: [self.date.timestamp]
         }
 
     def get_copyright(self) -> str:
diff --git a/src/test.db b/src/test.db
index 34facad66eae847b994d82c28a8b00d2de18594b..a7d8a2c13254cf3d0f00a49e73de18714088a673 100644
GIT binary patch
delta 137
ycmZozz|ydQWr8##=R_H2M$U~1OZd04F#h3Z<l4%}_@95$0~X@BB<X>f45I<~TP_Iz

literal 69632
zcmeI*-*4N-0S9pX@NXw(j35)d0mB6u78}mRqWD8(yRLH@CGh-fwWTiIx*13E&Sp$g
zu1IC|fNhA|1{g44?|mEg(w8B?_Sk=*Z+i{)w7+1_d)$$fY{{)NHeHqk{SvlBk;miR
z=Z>VvqmJ8eulX@2qTM-QG0_T_3dLgKM}!m#g{$&^R{r;gNjWmo{~+Is1LNn7UM<}G
z>5nfe|1Mk_{a0b;Ps-mfmS^JW=)#k!)#*FabCY+czB}>h<hRHFF!mQ200RUd009U*
zCxM4QxHMk5e7ShM6Ei2^zNg8**|)YD%gqL9F5g;f5IJFLeu~IJbl3q;M&!_J+-u&*
zMhT@vCoVhmV-}Er$Gj7fz~ASjdGFr6bjb%S=ngHJPL0~(?vC%JvD{j@yRy-|kxsv}
zxz$*?yFpeP@6PwaERe0noyJyU<E_Rv*_O-A``*IT!o&BJ@yfMp#pB<l$zKkf?!j^=
z_M>=U^wLmv24?iKMF>ll>!*YaCPmImqwCKco;R4#ejGz7O;?sAHOm+wLre5y9$qA!
znq}{JZf3l4^=k3M-=x_~@}1@GbW9n_-0Ad_yhQ`Zq?b3cvDqXWt+lm&vc4>nUkKz!
zJA$*g+eu6E*5>9~V|n8w6uI5Fv)o#1l4?KF!MyfTF_hCJvCnREcwxLUH&=Z4dRFpT
zp{K7Ghe$H89!IaT^%<61&CQ*a4H<sDvC$+y*;-j&-g-BwcUd07UVTq0`_#hX#j(np
zbH#!mdi;av{Xl+A+F{+eoqpcQ*0ZB#?=ugZQ)&7>ET`#PJL>pul)X+5r7xRyCVj~!
zkySPF-_jWK58~c-m~E8lCKRx6x65|Zl1ew#|Ir4wIQ=w(EH681_I}~v>Ue5!{C;ZC
zWSw0eXRi}O2HCVT208OD{NPM34_UmIh<q{My(n87D{r?NSrO;0q_X%YpO*D33p=%5
zRu(661Xi=algG~Jc%@P)eta#f%jsS$c^Mm0N_Qq_koqN`6D#)POsltCwioZ!7>5V_
z=!W#NJzEsZO+QKkoVpIyvf()%ljjfv!|5_^yBo%xqmy~3a~#%j+urG7r*qO2MD0$m
zqMsZn<Q^$Hl-prHvh(*zQws}YW0f~5=We~}5kO6e$B)BuT5mtDr{%HT?si<By^ai(
zNH*<DiX>-oL>;%E^i=g^XPVW`Zs)W>5{xsmUEYbAAI6ELBsrNouqT#t<iYtQ;2Tv!
ze@LX49}$lWKa{($;D|6HkI!<~7^N9aYeP06OWe7tYL?8wBTSa@ue@oEJZ)>qly}tZ
zee`&-I9|DOrTCHEE1hsRIg*slluWq$?B&wm?<e^`t%d$Tah*;Lc(|MGZ_-jozY|IG
z-ed{whvdKy0zcwzJM<zlFSmfeBMv9O>d162q~39dx%<5uep0=&4KF)pod??wGjwE4
zt8w-f;(BSU^7@tYs(k{Nz=peJ`2}+#`I322{+DO|Sx`PwzAxV}KmY;|fB*y_009U<
z00Izz00dr1f!pQr)|*Mn>!PX&(=n;Wb%*LTPovD$IjyU%Yck!b%enDhasnTb``sue
zENsVnydy7@guCyR##`5ABrRqOmmAEYauGpwoz<u#RGaFCuDg~H!l~AliE#J*a5rtZ
z5T>w&S+l66+b-4XnoDgiRO+fab9B3AFt`5eqD<prsn{|dmpdLesOK6rDg)TmR%L2U
z8Hx){bu34`!{rr*+ngUJR~lpj<ywMn?6<?#Hr-Z#Ms1r`<)D^6Ddk!Bl+0CGc~nrI
zD36rK2^0nhKmY;|fB*y_009U<00Izz00f?oz<jw?%yWUFoCM5s4We|t+?vdDIU#BP
zFDbJH<@fRh0|X!d0SG_<0uX=z1Rwwb2teSQBJe;q_0MfiOt!e^)GU*$R5Nu(b<Jdy
znU+aSt5)-vuA7`&2i?dQM|*L67`?W%6ofb9_Ms<kdVFbJ&T;8#hwbxlDbI;`SKb8h
zomIKssv%TUt#Vp(O^fQ>bf_(aNex3~nrUiAjj^M4H|{#TzvA@cO@}XeB8)tCKiJ#9
zzZZFVR-3f{mt@1gZ2w2=|2L(MBcBj}00bZa0SG_<0uX=z1R(IT3ncCTc`i((?f-eM
zBqZ(sMdi;0`G)}l5P$##AOHafKmY;|fB*y_0D%`L@bltyi!K%}C;jnH<oQkiX2{7M
zKa{;QFS|+40B-ky9Cq4XH~kdJ9t>4m-pEkZXjM(x|BK2K+5Z3HGKe)p00Izz00bZa
z0SG_<0uX=z1R(Hi0?F_H^IR*)`v3o|puDS0&3yK3>%vkHfB*y_009U<00Izz00bZa
zf#)c&ULI{FUGl50p>xBeLRGb-BmO#Nb*)OPhAyf#OS8<nk*9CH^->yLwdTpb{5?)(
z7yPE|;h(#dY4xh?wcl|KF1!7=>w2CZ_#S?*INIWFi;J2nYPLZg&XVT@2%S2bZct$x
zmg+IbuspGJz&)QW1*{V-y(7ExKj2HZd?)ae2N3LMT?s5Dk0H2uaH!=8``c`pyubfX
z`~QC|`~Sy10MD_yB5DXg00Izz00bZa0SG_<0uX>e+O2<%>lkVO|9P%Zq`&{qbLj!k
z|3mGA00bZa0SG_<0uX=z1Rwwb2>ibUUhMP#$}|5cD1TI5lW!Ox009U<00Izz00bZa
z0SG_<0<W0BFXhhzqS-=K^$e|6^{A%l@^N~WS(A^`tIDVPc~w`qhE->3P2Of7al$&B
zMC}7k){Z*98<BXA#l&le*JI-FK;C{&;x=L75!sgGJlvMQ7q~Lp(rt&CTt=wMPW7nn
z3WM6J#i(XDmTBu%Zd9F^M==lc+>YOtKN?t{Z5gV3@Ss+=WCwlBrMhVf%3N1|bWDT0
z!ZS=&Ka*&WA9i>Y`42b=()8zB;Ys`dBjqvf|9{1*IFbti2tWV=5P$##AOHafKmY=-
zm_XA0pXWkFdjEf(D-P-X|9P$=r2YRXK|%RM36#%YF*YO@0uX=z1Rwwb2tWV=5P$##
zAn+0j+!!slD%>@V8Z$gvH#I?ZuG!Qw9FN+bUUxm!6sGIk-D^kjjgkz_xNX&K*}0*d
zB|A5)dxBbw>$Ga>RfijlS%wxJ27YXgm0OFJ&8nI$J3!=ioy*$>EI}QiX|%=-&-2`x
z%T0ad`T_An!id`z!VjlpqkmcXMM3ud_gKDRfB*y_009U<00Izz00bZa0SG|gs|qB|
z`gzXxN0WegPTt3ofO)P7lw?xp^CUNE{~ww8yrBG5@sw}L5eyK300bZa0SG_<0uX=z
z1Rwx`msntJWTe%a%=4VTJm=YKWf|K<p4jr7-mjICwB`Bvf3DbyN!s$nmM5pl`F}|f
z1?6w@1p@>i009U<00Izz00bZa0SG_<0$)R5Ro=_r5~i?)S+l66+p<%?y5>@w3zfR6
z&K%vY8O*KcY1>~d=8P=w`G4Wk*T@<cg#ZK~009U<00Izz00bZa0SG|gr5E@Q;u*H?