[14:34:53] Yaron: ping [15:21:37] a3li: hello! [15:22:24] Yaron: hey, I'm having the same issue as you quickly looked at here: http://www.mediawiki.org/wiki/Extension_talk:Semantic_Forms/Archive_March_to_April_2013#Bug_with_preview [15:22:50] looking at the JavaScript handling preview, is it really a URL rewriting issue? [15:23:19] it uses wgTitle, not wgPageName, so if it's not in the main namespace, the name would be wrong anyway [15:31:26] a3li: what do you mean by "JavaScript handling preview"? [15:32:34] Yaron: the SF_preview.js file [15:52:13] Oh, I get it. [15:52:33] Sorry, I'm in and out of being on my computer today. [15:53:08] I think I would need more information to help you... I don't really know of anything related to this off the top of my head. [15:53:49] sure, anything you need :) [15:55:13] Well, what about that URL-related question I asked the other person? [15:55:48] it looks pretty much the same [15:55:59] sec [15:57:28] http://a3li.li/~alex/sf-preview1.png [15:58:17] same with special:formedit: http://a3li.li/~alex/sf-preview2.png (preview works here) [16:00:50] a3li: okay, cool. The lack of a "form" value in the non-working page certainly looks suspicious... [16:01:29] I assume there's no "form" value further down the list? [16:03:28] no [16:03:31] javascript confirms that [16:03:48] if wgAction == formedit, it only sets target (and that wrongly) [16:04:41] Ah! Now I get the issue. [16:05:30] I currently don't see a way the client javascript could know the form name [16:05:33] there's no value set anywhere [16:05:46] Right, it doesn't need the form name, apparently. [16:05:46] it suggests it will look up the form by page name, which in my case fails as the namespace is stripped [16:06:05] I didn't write the preview JS code (Stephan did), so I don't know all that much about it, although I assume I would have heard if it always failed for pages not in the main namespace... [16:06:12] Maybe that's really the bug, though. [16:06:43] Does it work if you change that variable from wgTitle to wgPageName, as you suggested? [16:06:48] hang on [16:08:22] seems to work after doing mw.config.set({"wgTitle": "GLEP:1"}); in the js console [16:11:22] Well... that's good to know, but not 100% useful. :) [16:11:41] Oh - I'm guessing you don't have access to modify the original code? [16:17:17] I do [16:17:26] but the effect should be the same [16:18:09] I suppose. [16:18:29] Okay, cool - I'll email Stephan and let him know about that issue. [16:18:34] Thanks for the detective work. [16:19:08] great, thanks [16:38:41] (PS1) Yaron Koren: Fixed handling for "action=formedit" URLs for non-main namespaces [extensions/SemanticForms] - https://gerrit.wikimedia.org/r/108335 [16:40:05] (CR) Yaron Koren: [C: 2 V: 1] Fixed handling for "action=formedit" URLs for non-main namespaces [extensions/SemanticForms] - https://gerrit.wikimedia.org/r/108335 (owner: Yaron Koren) [16:40:07] (Merged) jenkins-bot: Fixed handling for "action=formedit" URLs for non-main namespaces [extensions/SemanticForms] - https://gerrit.wikimedia.org/r/108335 (owner: Yaron Koren) [16:40:29] a3li: fixed! (I hope) [17:05:25] Yaron: cool, thanks [17:09:08] Looking now at those old talk page entries, that was clearly the issue back then as well. [17:09:16] Oh well.