[Cuis-dev] Bug removing and renaming a root system category
Hernan Wilkinson
hernan.wilkinson at gmail.com
Tue Feb 1 12:59:38 PST 2022
oh, BTW, the tests testRenamesWithoutLeadingBlanks
and testRenamesWithoutTrailingBlanks were failing because of the rename you
made from #renameCategory:toBe: to #renameCategory:to:. I just pushed the
fix.
Due to the loose coupling on Smalltalk, when renaming anything
(message,class, inst var, etc), we have to run the tests and inform to the
list because other packages may send those messages or reference the class
(in this case, I had to change DenotativeObjects for example :-) )
Cheers!
Hernan.
On Tue, Feb 1, 2022 at 5:47 PM Hernan Wilkinson <hernan.wilkinson at gmail.com>
wrote:
> Hey! no need to apologize! We are all human :-)
> I'll let you know if I find something else.
>
> The change set did the work. I pushed it to github
>
> Thanks!
> Hernan.
>
> On Tue, Feb 1, 2022 at 5:00 PM Luciano Notarfrancesco <luchiano at gmail.com>
> wrote:
>
>> Hi Hernan,
>> This should fix it. Sorry, I don't use all of the features of the browser
>> so I haven't noticed this problem, and there might be more... let me know
>> if you find any other issues.
>>
>> On Tue, Feb 1, 2022 at 5:21 PM Hernan Wilkinson via Cuis-dev <
>> cuis-dev at lists.cuis.st> wrote:
>>
>>> Hi Luciano!
>>> I just found that removing a system category does not work. If you are
>>> in one of the categories it does, but it does not when in the root
>>> category...
>>> I think it would be great to remove all subcategories if you remove the
>>> root category.
>>>
>>> The same thing happens when renaming a root category.
>>>
>>> Cheers!
>>> Hernan.
>>> --
>>> Cuis-dev mailing list
>>> Cuis-dev at lists.cuis.st
>>> https://lists.cuis.st/mailman/listinfo/cuis-dev
>>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20220201/2ee280ec/attachment.htm>
More information about the Cuis-dev
mailing list