[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-11-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

Andre Klapper aklap...@wikimedia.org changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #13 from Andre Klapper aklap...@wikimedia.org ---
No open patches to review here (backport patch to REL_1_21 got merged), hence
resetting status to RESOLVED FIXED. Backport_to_Stable flag might be set to +
by hexmode.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-11-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

Gerrit Notification Bot gerritad...@wikimedia.org changed:

   What|Removed |Added

 Status|RESOLVED|PATCH_TO_REVIEW
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-11-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

--- Comment #11 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 95391 had a related patch set uploaded by MarkAHershberger:
(bug 41769) Add frame:callParserFunction() and frame::extensionTag()

https://gerrit.wikimedia.org/r/95391

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-11-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

--- Comment #12 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 95391 merged by MarkAHershberger:
(bug 41769) Add frame:callParserFunction() and frame::extensionTag()

https://gerrit.wikimedia.org/r/95391

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-04-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

--- Comment #9 from Brad Jorsch bjor...@wikimedia.org ---
(In reply to comment #8)
 
 I take that to mean the change is *not* supposed to be released as part of
 1.21.
 Fine with me.

Take it more literally: If 1.21 should be compatible with a version of
Scribunto later than 0dc45ff2f89b8a6b90cee7e0c1fb70e2a38985f1, then this would
need to be backported.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-04-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

--- Comment #10 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Related URL: https://gerrit.wikimedia.org/r/59637 (Gerrit Change
I339b882010dedd714e7965e25ad650ed8b8cd48f)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-04-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

PleaseStand pleasest...@live.com changed:

   What|Removed |Added

 CC||pleasest...@live.com

--- Comment #6 from PleaseStand pleasest...@live.com ---
Is gerrit 52060 necessary for Scribunto compatibility with MW 1.21? If so,
should it be backported?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-04-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

--- Comment #7 from Brad Jorsch bjor...@wikimedia.org ---
(In reply to comment #6)
 Is Gerrit change #52060 necessary for Scribunto compatibility with MW 1.21?

Only if you want to use a version of Scribunto later than
0dc45ff2f89b8a6b90cee7e0c1fb70e2a38985f1.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-04-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

PleaseStand pleasest...@live.com changed:

   What|Removed |Added

   Target Milestone|--- |MW 1.22 version

--- Comment #8 from PleaseStand pleasest...@live.com ---
(In reply to comment #7)
 (In reply to comment #6)
  Is Gerrit change #52060 necessary for Scribunto compatibility with MW 1.21?
 
 Only if you want to use a version of Scribunto later than
 0dc45ff2f89b8a6b90cee7e0c1fb70e2a38985f1.

I take that to mean the change is *not* supposed to be released as part of
1.21.
Fine with me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-03-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

Brad Jorsch bjor...@wikimedia.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Brad Jorsch bjor...@wikimedia.org ---
Merged. Note this will likely be deployed along with 1.22wmf1, see
https://www.mediawiki.org/wiki/MediaWiki_1.22/Roadmap for the schedule.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-03-04 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

Brad Jorsch bjor...@wikimedia.org changed:

   What|Removed |Added

 CC||bjor...@wikimedia.org
   Assignee|wikibugs-l@lists.wikimedia. |bjor...@wikimedia.org
   |org |

--- Comment #4 from Brad Jorsch bjor...@wikimedia.org ---
Gerrit change 52060
Gerrit change 52061

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-03-04 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

Brad Jorsch bjor...@wikimedia.org changed:

   What|Removed |Added

   Keywords||patch-in-gerrit

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-02-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

Yannick Perret yper...@in2p3.fr changed:

   What|Removed |Added

 CC||yper...@in2p3.fr

--- Comment #1 from Yannick Perret yper...@in2p3.fr ---
As far as I know parser functions are not real templates (but it uses the
same syntax so...).

Calling fullurl works if you use frame:preprocess() rather that
frame:expandTemplate().

Whatever in my opinion all these functions should be mapped inside mw module,
including pseudo-templates such as {{PAGENAME}} and others.

Regards,

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-02-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

--- Comment #2 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com ---
(In reply to comment #1)
 As far as I know parser functions are not real templates (but it uses the
 same syntax so...).
 
 Calling fullurl works if you use frame:preprocess() rather that
 frame:expandTemplate().

frame:preprocess() does not handle argument data directly. It's far to
unreliable to use when you have user specified data. And iirc with what I
remember of the way our parser works with it's handling of variables, it's
actually impossible to use frame:preprocess in every situation that
frame:expandTemplate would work.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-02-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

--- Comment #3 from Yannick Perret yper...@in2p3.fr ---
That's true. It can be used as a workaround but not for general use.

Whatever I still think that the best way is to treat these features at Lua/C
level in order to avoid calling preprocess/expandTemplate.
It is what is expressed in
http://www.mediawiki.org/wiki/Extension:Scribunto/Deployment_priorities#Should_have
(the goal of Lua is to avoid calling too much mediawiki preprocess stuff, after
all).

Something like mw.parserfunctions maybe, or something with dedicated submethods
such as mw.page, mw.tools, ...

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 41769] frame:expandTemplate doesn't work with parser functions

2013-02-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=41769

Andre Klapper aklap...@wikimedia.org changed:

   What|Removed |Added

   Priority|Unprioritized   |Normal

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l