On Wed, Aug 9, 2017 at 4:39 AM, Herby VojÄÃk <herby@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