picard-plugins icon indicating copy to clipboard operation
picard-plugins copied to clipboard

pr fixing issue exception when artist sorted name has no last name pa…

Open sambhavnoobcoder opened this issue 1 year ago • 3 comments

…rt issue#350

fixes #350

added conditional , that recognises a single word artist name different from one with a last name .In such cases, the code generates initials for this single word to serve as an abbreviated representation.The initials are formed by taking the first letter of each part of the single word and appending a dot ('.') after each letter.

moreover the code also handles the whitespaces in sorted and unsorted names .

sambhavnoobcoder avatar Jan 02 '24 11:01 sambhavnoobcoder

okay , I'll take this into account and redraft smaller pr's in a systematic manner to make it easier to review and more step by step to resolve any issues in the code as well . in doing so , I'll also provide with the adequate testing use cases for both before and after the changes to assist with testing the changes in the code . thanks .

sambhavnoobcoder avatar Jan 02 '24 13:01 sambhavnoobcoder

@sambhavnoobcoder Yes, that would be great. I find this a bit difficult to review as it is, but you already had to wrap you head around all the functionality here. Especially some real test cases, if you have some, would be useful.

phw avatar Jan 05 '24 17:01 phw

I would love to see this get fixed, having quite a few errors with this plugin.

eharris avatar May 19 '24 16:05 eharris