[01:31:16] TimStarling: reminder re: https://gerrit.wikimedia.org/r/#/c/198661/ , I reviewed it already but you'll probably pick out different things. [13:01:23] tgr|away: hook, yes. password handling also updated the commit message. AuthManager added some stuff. [14:09:21] tgr|away: (although the "added stuff" in AuthManager is a feature that was added in the password handling patch with the intention that it be redone in AuthManager and I had forgotten to actually do so when I first split that out) [20:25:13] anomie: have a minute for a question about html form cloner? [20:25:35] csteipp: In a minute? [20:25:51] Sure! [20:32:51] csteipp: ok [20:33:01] * anomie is ready now [20:33:30] Cool :). So I'm using a cloner to allow someone to add fields to a configuration [20:33:40] Creating the config works great [20:33:50] But I also want to allow them to edit an existing on [20:33:51] one [20:34:27] And haven't figured out how to tell htmlform to add bunches of those field groups that were originally cloned. [20:34:39] Is there somewhere that documents how to do that? [20:36:56] csteipp: There probably isn't documentation anywhere. Are you trying to supply the data-to-be-edited by setting 'default' in the field definitions or by setting ->mFieldData on the HTMLForm? [20:37:32] I was just setting 'default' [20:39:13] anomie: Actually, no, neither. [20:39:42] Oh? What are you doing then? [20:41:54] anomie: I only got as far as trying to figure out what fields to pass to new HTMLForm-- and couldn't figure out a way to pass multiple versions of one 'field' (that had previously been cloned) into HTMLForm [20:42:17] I suppose the data structure you get back from the cloner field is something like array( array( 'field1' => 'A', 'field2' => 'B' ), array( 'field1' => 'C', 'field2' => 'D' ) )? Try setting exactly that as the default for the cloner field. [20:45:47] Hmmm.. Exception: Descriptor with no class for wpparams[HTMLFormFieldCloner1x] [20:47:37] That was passing an array of field arrays in 'fields' to the cloner [20:48:22] You shouldn't have had to touch 'fields' to add a default. [20:51:25] Oh, that sortof works! [20:51:46] Undefined index: id in /home/csteipp/code/core/includes/htmlform/HTMLFormFieldCloner.php [20:52:24] I have an id field in the cloner, since I thought it was required... do I actually need it? [20:52:48] No, appears not [20:53:55] anomie: Thanks, that seems to work [20:54:00] csteipp: You're welcome [21:01:22] anomie: My initial thought on SessionManager is that sounds like a good direction. [21:59:04] tgr: off the top of your head, do you remember if it is possible to replace the '--config=/path/to/config/file' argument to the sentry CLI binary with SENTRY_CONF=/path/to/config/file env var? [21:59:37] I know that you do that for the systemd unit files, but wasn't sure if the same could be done for the other invocations [22:06:57] ori: never tried but I don't know of any reason why it wouldn't work [22:07:46] should I change that and do a test run? [22:08:36] tgr: 'sokay, i'll try it out [22:39:15] I'm getting this: exception 'Less_Exception_Chunk' with message 'ParseError: Unexpected input in watchstar.less on line 1, column 0 [22:39:15] after updating to latest master. Anybody knows what's going on? [22:42:32] ori: any ideas ^? (since you were the one replacing the lessphp dep :) [22:44:42] nm, found the solution :) [23:46:53] SMalyshev: update vector, right?