Bug in OpenRefine partion() documentation?

9 views
Skip to first unread message

fabius pocus

unread,
Oct 23, 2022, 3:57:37 PM10/23/22
to OpenRefine
This is a piece of text of partition() documentation:

partition(s, s or p fragment, b omitFragment (optional))

Returns an array of strings [ a, fragment, z ] where a is the substring within s before the first ..............................The expresion "abcdefgh".partition(/c.e/) will output [“abc”, "cde", defgh” ].


I think that:

"abcdefgh".partition(/c.e/)

[ "ab", "cde", "fgh" ]

and not:

[“abc”, "cde", defgh” ]


Fabio


Owen Stephens

unread,
Oct 24, 2022, 7:31:19 AM10/24/22
to OpenRefine
Thanks Fabio - I've raised an issue on Github and submitted a fix for review https://github.com/OpenRefine/OpenRefine/issues/5371

Tom Morris

unread,
Oct 24, 2022, 12:17:46 PM10/24/22
to openr...@googlegroups.com
Thanks Owen!
Fabio - you can do this yourself too. The issue tracker is unrestricted and available for everyone to use.

Tom

--
You received this message because you are subscribed to the Google Groups "OpenRefine" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openrefine+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/openrefine/81bb356e-f2fe-4503-a6b7-03c7f791344dn%40googlegroups.com.
Reply all
Reply to author
Forward
0 new messages