Le 28/5/15 01:56, Sergio Fedi a ��crit��:
Hi list!

As we are adding comments to Packages in Pharo, we stumbled upon the fact that there are other creatures present in the package list.

These are:

RPackageTag, which models the tags under the package
DynamicGroup, which model the dynamic categories of LastModifiedClasses, MostViewedClasses and Work

Objects of these classes WILL have to understand the accessor #comment

1 - How do we want RPackageTag and DynamicGroup to handle #comment and #comment: ?
2 - Are there any other classes present in that list we are not aware of?

Some options we can consider:

a - Hardcode in an generic answer to #comment, do nothing on #comment, leave them be for now

I would just return an empty string.
I think that the problem is that the item list elements should be customized to not send the messages comment to the
wrapped items (Tag, DynamicGroup....).

Stf

b - Have the object behave as a RPackage, it can have a persistent comment. In this case, where do we save that data?

Since the Package Comments case is taking so long, and to keep the scope of the case in check, we are incline to handle both classes with option a.

But we wanted to check with the list first, just in case.

Ok, now let's hear you feedback.