[21:40:15] <wm-bb>	 <hecko_yes> i'm not sure if this is the place for this but i'm getting a curious error when i try to do recursion via composition
[21:40:15] <wm-bb>	 <hecko_yes> specifically `object of type 'ZObject' has no len()` despite none of the functions used ever calling `len()`
[21:40:16] <wm-bb>	 <hecko_yes> see https://wikifunctions.beta.wmflabs.org/wiki/Z10381
[21:48:54] <wm-bb>	 <vrandecic> You're on the right place, let me take a look
[21:51:57] <wm-bb>	 <vrandecic> I can replace the error message. Need to finish something up and then I'll take a close look
[21:58:12] <wm-bb>	 <vrandecic> One thing that happens to me all the time (we are working on a better UI to help with that): right now you're using the argument and putting it into a positive integer. But your argument already is a positive integer. You need to use the argument one level up.
[21:58:56] <wm-bb>	 <vrandecic> I replaced it already in the first usage of subtrahend, within the call to equality in the if condition
[21:59:07] <wm-bb>	 <vrandecic> In order to show what i mean
[21:59:43] <wm-bb>	 <vrandecic> I don't know if that will fix all problems, but it'll take us a step closer
[22:00:30] <wm-bb>	 <vrandecic> Now there are two more things that make it hard right now: first, the history doesn't show the diff yet, so you can't tell what I changed (we're working on it)
[22:01:52] <wm-bb>	 <vrandecic> Second, when you change the implementation, the publish button remains grayed out, because that's not noticed. That's a bug, needs to be filed.  A workaround is to make an edit to the label or something, you can redo it before publishing, but it allows to safe.
[22:02:59] <wm-bb>	 <vrandecic> If you want to try to fix the problem I mentioned above, I'm happy to wait and then check afterwards if there are still problems, but if you prefer I can also make these changes and then we take it from there - whatever you prefer
[22:04:45] <florent>	 <wm-bb> "<vrandecic> It would be good..." <- I reported this ticket: https://phabricator.wikimedia.org/T325412
[22:05:19] <wm-bb>	 <vrandecic> Thank you!
[22:08:11] <florent>	 Funny thing about something else: I was about to report a bug about the fact that the first time we enter space on a cdx-lookup field, the space is not taken into account, but hopefully Dani has already taken care of it and fixed it :)
[22:08:11] <florent>	 https://github.com/wikimedia/codex/commit/4342cb3c9faecaf47d4762488227633b6fa8afc1
[22:09:46] <wm-bb>	 <vrandecic> Sweet! I think I also stumbled on that one and filed it a week or two ago.
[22:09:50] <wm-bb>	 <vrandecic> Thank you!
[22:12:26] <florent>	 (I was about to make some weird workaroud for selenium tests, but I found a much simpler one until the codex dependency is updated ^^)