IPB

Welcome Guest ( Log In | Register )

 
Reply to this topicStart new topic
> [F] $validate() bug?, Does not consider folder paths (only filename)
blackspawn
post Jan 21 2010, 17:16
Post #1


Member


Group: Full Members
Posts: 25
Joined: 1-October 05
Member No.: 2192
Mp3tag Version: 2.46d



I'm using version 2.45a in Vista SP2, and the $validate() function is only validating the filename, not the full path. A short example:

$validate(%album%\%artist%, '_') only evaluates %artist% for invalid characters, if there are any present in %album% they just get removed.
However, $validate(%album% %artist%, '_') validates everyting (album and artist)

The help file cleary states: Remove invalid characters from file name/file paths so this should work for file paths too and not only for the filename right?

I was using this with 2.44 and it worked ok, should I be using another syntax for validating the path AND filename?
Go to the top of the page
 
+Quote Post
Florian
post Jan 21 2010, 21:40
Post #2


Developer


Group: Admin
Posts: 7706
Joined: 12-December 01
From: Germany, Dresden
Member No.: 203
Mp3tag Version: 2.62



If I have ARTIST=Test? and ALBUM=Test? and use $validate(%album%\%artist%,_) I get Album_\Artist_ as result.

Can you please post detailed information about the content of the tag fields and the result you're observing.

Thanks!

Florian


--------------------
♫ If you like using Mp3tag please donate to support further development.

Go to the top of the page
 
+Quote Post
blackspawn
post Jan 22 2010, 19:20
Post #3


Member


Group: Full Members
Posts: 25
Joined: 1-October 05
Member No.: 2192
Mp3tag Version: 2.46d



Hi Florian, thx for the quick reply. Here is the info you requested:

Tag-Filename code

$validate($if($eql(%genre%,'Soundtrack'),'_Soundtracks\['%year%']' %album%\,%artist%\'['%year%']' %album%\)$ifgreater($right(%discnumber%,1),1,CD$left(%discnumber%,1),)\$num(%track%,2). %title%,'_')


Test case

Artis: "Iced Earth"
Album: "The Crucible of Man: Something Wicked Part 2"
Title: "Sacred Flames"
Year: "2008"
Track: "1"
Disc: "1/1"
Tag: "Flac"


Result

Iced Earth\[2008] The Crucible of Man Something Wicked Part 2\01. Sacred Flames.flac



.... also a simple $validate(%album%\%title%, '_') has the same result regarding the validation of the album string, however if I use $validate(%album% %title%, '_') the %album% part is correctly evaluated.

This post has been edited by blackspawn: Jan 29 2010, 10:40
Go to the top of the page
 
+Quote Post
blackspawn
post Feb 21 2010, 17:24
Post #4


Member


Group: Full Members
Posts: 25
Joined: 1-October 05
Member No.: 2192
Mp3tag Version: 2.46d



Hi Florian,

I don't mean to bother you but have you any news or been able to verify this on your end?


Thanks!
Go to the top of the page
 
+Quote Post
Florian
post Feb 21 2010, 21:18
Post #5


Developer


Group: Admin
Posts: 7706
Joined: 12-December 01
From: Germany, Dresden
Member No.: 203
Mp3tag Version: 2.62



Yes, it happens for colons only and will fixed with the next version. Thanks for reminding me smile.gif

Kind regards,
Florian


--------------------
♫ If you like using Mp3tag please donate to support further development.

Go to the top of the page
 
+Quote Post
blackspawn
post Feb 21 2010, 23:41
Post #6


Member


Group: Full Members
Posts: 25
Joined: 1-October 05
Member No.: 2192
Mp3tag Version: 2.46d



QUOTE (Florian @ Feb 21 2010, 20:18) *
Yes, it happens for colons only and will fixed with the next version. Thanks for reminding me smile.gif

Kind regards,
Florian


Your just awesome! :-) Thanks!
Go to the top of the page
 
+Quote Post
Florian
post Feb 27 2010, 15:12
Post #7


Developer


Group: Admin
Posts: 7706
Joined: 12-December 01
From: Germany, Dresden
Member No.: 203
Mp3tag Version: 2.62



This is now fixed with Mp3tag v2.45c.


--------------------
♫ If you like using Mp3tag please donate to support further development.

Go to the top of the page
 
+Quote Post

Reply to this topicStart new topic
1 User(s) are reading this topic (1 Guests and 0 Anonymous Users)
0 Members:

 



RSS Lo-Fi Version Time is now: 21st August 2014 - 18:59