[Pharo-users] TextAction bugs

Nicolai Hess nicolaihess at gmail.com
Mon May 16 07:42:16 EDT 2016


2016-05-16 8:29 GMT+02:00 Alistair Grant <akgrant0710 at gmail.com>:

> Hi All,
>
> TextAction appears to have a couple of issues:
>
>
> TextAction>>actOnClick: anEvent for: anObject in: paragraph editor: editor
>         "sent when a user clicks on a piece of text to which I am applied
> in an editor"
>
>         "may be self is included in the event or an Object. "
>         ^ actOnClickBlock cull: self cull: anEvent cull: anObject cull:
> paragraph cull: editor
>
>
> #cull:cull:cull:cull:cull: doesn't exist, so should be something like:
>
>
> TextAction>>actOnClick: anEvent for: anObject in: paragraph editor: editor
>         "sent when a user clicks on a piece of text to which I am applied
> in an editor"
>
>         "may be self is included in the event or an Object. "
>         ^ actOnClickBlock valueWithEnoughArguments: (Array with: self
> with: anEvent with: anObject with: paragraph with: editor)
>
>
> It is also missing the Rubric version:
>
>
> TextAction>>rubActOnClick: anEvent for: anObject in: paragraph editor:
> editor
>         "sent when a user clicks on a piece of text to which I am applied
> in an editor"
>
>         "may be self is included in the event or an Object. "
>         ^ actOnClickBlock valueWithEnoughArguments: (Array with: self
> with: anEvent with: anObject with: paragraph with: editor)
>
>
> I've followed the convention in the other implementations of
> #rubActOnClick:for:in:editor: and reimplemented the code, but it could
> just as easily call #actOnClick:for:in:editor:.
>
> Any reason not to submit this as a formal bug report?
>
> Should it be submitted as two separate bug reports since they are in
> separate packages?
>

There is already a bug report for the first issue you described:
18200
<https://pharo.fogbugz.com/f/cases/18200/TextAction-calls-unimplemented-method-cull-cull-cull-cull-cull>
TextAction calls unimplemented method #cull:cull:cull:cull:cull:



>
> Thanks!
> Alistair
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.pharo.org/pipermail/pharo-users_lists.pharo.org/attachments/20160516/5489abe7/attachment.html>


More information about the Pharo-users mailing list