GNOME Bugzilla – Bug 736610
"Process Fields | Convert:" not working as expected
Last modified: 2014-11-18 09:08:10 UTC
Created attachment 286123 [details] "Process Fields | Convert:" issue (Before) I have a group of MP3's that are named something like these: 04 - Like To Get To Know You (Single Version) - Spanky & Our Gang.mp3 05 - Lazy Day (Album Version) - Spanky & Our Gang.mp3 I want to remove " (Single Version)" and " (Album Version)" in each place that it occurs in a Title tag field. With the current version of Easytag from Git Master, I cannot do this. Here is a screenshot of the MP3's that I am attempting to edit in this manner: http://www.afolkey2.us/gallery3/var/albums/Assorted/Easytag%20Screenshot%20001.png?m=1410600702 Now here is a screenshot of those same MP3's after I attempt to remove the "(Album Version)" string from all the title fields: http://www.afolkey2.us/gallery3/var/albums/Assorted/Easytag%20Screenshot%20002.png?m=1410600673 You will notice that it looks like both screenshots are the same. That's because nothing happens when I click "Scan Files" If you would rather have those images as attachments to this bug, I will add them there as well.
Created attachment 286124 [details] "Process Fields | Convert:" issue (After)
Also, I currently have "Display changed files in list using:" set to Bold. So obviously, if any of these sample files had been modified when I clicked "Scan Files" it would show up in the second screenshot, even though all the files are still selected.
This is because the conversion feature uses Perl-compatible regular expressions, and the parentheses have special meaning there, so must be escaped. If you use a backslash before the brackets, you will see that the conversion works as expected (for example, " \(Single Version\)"). This is a bit awkward for simple strings, but very powerful for more complex matches. It might be a good idea to provide a simple "text-only" conversion, as a separate feature, but I have also added a task to the Etherpad to improve the documentation around this part of the scanner.
*** Bug 740284 has been marked as a duplicate of this bug. ***
Please change this, make it optional (add extra field) or at least add a comment that the user has to use regular expressions. I doubt that most users can handle regexps.