On Wed, Aug 9, 2017 at 4:39 AM, Herby Vojčík <he...@mailbox.sk> wrote:

> Hello!
> I've got this portion in my delegate:
>         requestPayload ifNotNil: [ uuidKeys do: [ :each |
>                         requestPayload at: each ifPresent: [ :s |
> requestPayload at: each put: (UUID fromString: s) ] ] ].
>         responsePayload := self towergame clientSync: requestPayload.
>         responsePayload ifNotNil: [ uuidKeys do: [ :each |
>                         responsePayload at: each ifPresent: [ :uuid |
> responsePayload at: each put: uuid asString ] ] ].
> Now I would gladly use something like Dictionary >> at:ifPresentTransform:
> aBlock. But it is not present, so I have two choices:
>   1. Add it as extension method, but then it may clash if someone else has
> similar idea.
>   2. Add private helper TowergameDelegate >> dict:at:ifPresentTransform:,
> which is longer and needs additional self receiver.

3. Contribute an implementation into Pharo, which bypasses the problem of
clashing with someone else's package, when they have a similar need they'll
benefit from it already been done.  That is assuming such would be a
generally useful feature. It sounds like it to me, but could someone else
support that?  Keeping in mind its good to encourage newcomers to step up
to being a contributor and not just a user - expanding the circle of "Pharo
Is Yours"

Note though, this will probably only be integrated to Pharo 7.  So you'll
need your Baseline to conditional load it for Pharo 5/6 - but thats a good
thing to practice anyway (someone else will need to advise how to go about
it - I'm still getting up to speed on this)

cheers -ben

Reply via email to