[02:32:21] Greetings, [02:32:21] Import from flickr or other big-tech photo sharing platforms is permitted if the source licence is compatible with BY-CC-SA 3.0 or 4.0. [02:32:22] Is this tha case for personal blogs for infographics or illustrations? [02:32:22] Well regards, [02:32:23] mugli [02:32:57] Sorry, wrong room. [02:33:04] :( :) [02:33:32] (a very general answer would be that the license is usually more important than the source) [02:37:04] Thank You @Shaun for your swift answer! [02:37:30] 🤝 [07:11:14] $wgInvalidUsernameCharacters = ''; [07:11:42] I am trying to allow @ as a character in the usernames, however even with that line it complains about using @ in a username when creating a new user [07:11:49] Is that the wrong way to do it? [07:13:20] ah nvm [17:04:53] Hi, can someone point me as to how safely do dbw transactions in an extension? I've tried with doAtomic but I'm still having issues with corruption :/ [17:06:25] You might want to pastebin your code or link :) [17:10:23] nice, just a sec [17:12:08] here: https://share.riseup.net/#a6dvMiTWX10hSRCkavX1vw [17:13:13] fwiw, you can do an IN by just doing [ 'colname' => $arrayOfItems ] [17:15:02] cool, didn't know that [17:16:00] underneath it'll do an = if it's singular, IN if it's multiple [17:20:43] Ah, I forgot to say that this an extension developed as a SpecialPage. [17:20:48] So what do you mean by corruption? [17:22:00] It deletes the old revisions as expected, but some of the pages, dozens of them, throw an Exception when I try to read them [17:22:36] the exception is: MediaWiki\Revision\RevisionAccessException [17:24:00] Whats the error message etc? [17:26:02] I get: Fatal Exception of type "MediaWiki\Revision\RevisionAccessException" [17:26:33] and on the logs I can see "Failed to load data blob from tt:1010: Unable to fetch blob at tt:1010" in the trace [17:30:32] So I'd imagine that means there's nothing wrong with the transaction specifically, but you're not cleaning up everything you might need to [17:31:55] So the use of startAtomic/endAtomic pairs is correct? [17:33:04] this extension was using begin()/commit() first, but from what I understood from the docs, that is not safe for inner tx, right? [17:37:25] it looks correct to me [17:38:49] nice! thanks Reedy and legoktm [17:39:31] I go after this bug in other places then. [17:41:08] other places in the code, I mean :-)