Jump to content

Extension talk:Winter

Add topic
From mediawiki.org
Latest comment: 12 years ago by Danwe in topic Security and Winter
The following discussion has been transferred from Meta-Wiki.
Any user names refer to users of that site, who are not necessarily users of MediaWiki.org (even if they share the same username).

Improving and Extending Winter

[edit]

Winter is still in active development. If you spot any bugs or have ideas to add any new features or make other improvements, please feel free to comment.

addFunction method

[edit]

New functions can be added via the hooking method addFunction provided by the Winter parser

$Winter->addFunction('Winter_function_name','PHP_function_to_call');

PHP_function_to_call will take a single parameter: an array of the parameters of the Winter function. The value of the 0 index is the name of the calling function (e.g. 'Winter_function_name'). This allows for the same PHP function to handle multiple hooks for Winter functions.


Known bugs

[edit]

v2.1.1

[edit]

hello i'm using MW-1.12 and i'm getting this error

Notice: Undefined index: SCRIPT_URL in /opt/mediawiki-1.12.0-ZCUEdition/extensions/Winter.php on line 121
Notice: Undefined index: SCRIPT_URI in /opt/mediawiki-1.12.0-ZCUEdition/extensions/Winter.php on line 120

seems that problem is in these lines

$this->scriptVars['_system']['uri'] = $_SERVER['SCRIPT_URI'];
$this->scriptVars['_system']['url'] = $_SERVER['SCRIPT_URL'];

after commenting them everythink works fine (i think)


It seems that Winter 2.1.1 inhibits templates from parsing particularly long pieces of data. We use Winter on WikiPaltz (http://www.wikipaltz.com) and it's like a dream come true. However, after upgrading to 2.1.1, certain templates ceased to function when called with long strings of text. An example can be found here:

The "WriteLeg" template simply stopped showing with this code with Winter 2.1.1. Thus, we had to revert to 2.1.0. -RadicalHarmony on WikiPedia, JustinHolmes on WikiPaltz

v1.0

[edit]

All bugs below from v1.0 have been fixed in v1.1

  • nested functions with spaces inbetween the function name and the first pipe cause a time out error. ex: {#var | a {{#ifeq | a | b | c | d}} }} . a work around for this bug is to omit the space. The followin code works {#var| a {{#ifeq| a | b | c | d}} }} This bug was dure to an error in a regular expression and has been fixed for later versions
  • the operator != does not always work as expected. Current work around is to avoid using this operator. This error was caused because the mediawiki parser prepends an &nbps; to ! if there is a space inbetween it and a |

[2 bugs added 05:01, 18 November 2005 (UTC)]

Questions

[edit]
  • I tried to run the extension on Mediawiki 1.5.3. It seemed to work, until i tried nested functions like the Example 1 from the mainpage. I always get Error messages about Timouts. Now the server is killed - though i am not sure, if the extension has something to do with it, but it seems so. Any ideas, what i can do about it? mail: tiefling at gmx.de -- Phonkeal
strange.. I haven't tested it on 1.5.3 but it works fine on 1.5.2 . You copied example 1 directly and it's not working? Frantik 23:33, 17 November 2005 (UTC)Reply
Your email revealed a bug in Winter code.. it doesn't like it if you put a space in between the function name and the first pipe (|). This has been fixed for future releases Frantik 05:01, 18 November 2005 (UTC)Reply

How to enable this extension?

[edit]

I'v dropped it into extensions directory, but it does not seem to work. It's mediawiki 1.5.2.

you need to add require_once("extensions/Winter.php"); to the bottom of LocalSettings.php

Small improvements

[edit]

Hello, your extension ist very good. But I have a few small improvements still.

  1. It could be contained version information
  2. The function "wfWinterScriptExtension" needs a return value, since otherwise different extension could not fade in itself into the hook.

to 1. the code

// Process page text using Winter scripting engine
function wfWinterScriptExtension( &$parser, &$text)
{  global $Winter;

   $text = $Winter->processText($text);

   return true;
}

to 2. the code

$wgExtensionCredits['parserhook'][] = array(
                                    	    'name' => 'Winter (Wiki INTERpreter)',
	                                    'author' => 'frantik',
	                                    'url' => 'http://meta.wikimedia.org/wiki/User:Frantik/Winter',
                                            'description' => 'Is an extension which adds an interpreted language to Mediawiki pages.',
                                            'version' => '1.3.1');

--Thomas Klein 18:40, 11 December 2005 (UTC)Reply

Cheers for the advice. I will include these changes in the next update. Glad you like the extension :) --Frantik 14:23, 14 December 2005 (UTC)Reply

Problems with tables in templates

[edit]

I have a template with a table. So lines in the table don't include in the table, if the parameter is null.

Template source:

{| border="0" cellpadding="2" cellspacing="1" style="background:#8CAFE6; float:right; margin-left:15px; margin-top:10px; margin-bottom:10px" width="30%"
! colspan="2" align="left" | Mitarbeiter
|---- style="background:#FAFAF5"
|Name:||'''{{{Name}}}'''
|---- style="background:#FAFAF5"
|Tel:||{{{Telefon}}}
|---- style="background:#FAFAF5"
|{{#default|Arbeitsplatz|}}
{{#if| {{{Arbeitsplatz}}} | 
|Arbeitsplatz:||{{{Arbeitsplatz}}} 
|---- style="background:#FAFAF5" }}
{{#default|Besonderheit|}}
{{#if| {{{Besonderheit}}} | 
|Besonderes:||{{{Besonderheit}}} 
|---- style="background:#FAFAF5" }}
|}

Template call:

{{Mitarbeiter|Name = Thomas Klein
|Telefon = +49 111/555-888
|Besonderheit = The line Arbeitsplatz is delete, because the parameter is missing
}}

What is wrong? --Thomas Klein 21:12, 11 December 2005 (UTC)Reply

You need to tell Winter the default values of any value that may be NULL. Include this text at the top of your template page:
{{#default | Arbeitsplatz | }}
{{#default | Besonderheit | }}
--Frantik 14:18, 14 December 2005 (UTC)Reply
That works!
{{#default | Besonderheit | }}
{{#if| {{{Besonderheit}}} | 
|Besonderes:||{{{Besonderheit}}} }} 
|---- style="background:#FAFAF5" 
|}
But not that,
{{#default | Besonderheit | }}
{{#if| {{{Besonderheit}}} | 
|Besonderes:||{{{Besonderheit}}} 
|---- style="background:#FAFAF5" }}
|}
and not that
{{#default | Besonderheit | }}
{{#if| {{{Besonderheit}}} | 
|Besonderes:||{{{Besonderheit}}} 
{{#if| {{{Besonderheit}}} | 
|---- style="background:#FAFAF5" }}
|}

Why not? --Thomas Klein 21:56, 14 December 2005 (UTC)Reply

It seems as if not all of the code is passed to Winter if it is inside a table. I'll try to see if I can make this work as I'm sure it would be a popular feature --Frantik 23:02, 14 December 2005 (UTC)Reply
Ok, I see the problem. The closing }} for the if statement needs to be on a new line, otherwise mediawiki assumes it is part of the table definition. Try this:
{{#default|Arbeitsplatz|}}
{{#default|Besonderheit|}}
{| border="0" cellpadding="2" cellspacing="1" style="background:#8CAFE6; float:right; margin-left:15px; margin-top:10px; margin-bottom:10px" width="30%"
! colspan="2" align="left" | Mitarbeiter
|---- style="background:#FAFAF5"
|Name:||'''{{{Name}}}'''
|---- style="background:#FAFAF5"
|Tel:||{{{Telefon}}}
|---- style="background:#FAFAF5" 
{{#if| {{{Arbeitsplatz}}} | 
|Arbeitsplatz: ||{{{Arbeitsplatz}}}
|---- style="background:#FAFAF5" 
}}
{{#if| {{{Besonderheit}}} |
|Besonderheit: ||{{{Besonderheit}}}
|---- style="background:#FAFAF5" 
}}
--Frantik 06:30, 15 December 2005 (UTC)Reply
That is ok, but not if i use the varibale more than one. --Thomas Klein 07:53, 16 December 2005 (UTC)Reply
What do you mean? Can you give an example? --Frantik 11:23, 17 December 2005 (UTC)Reply

nowiki is not honored!

[edit]

This is really a minor nit -- so far, I love Winter! But I was making an example page for my users, showing entered versus displayed and it appears that Winter merrily cruises right through <nowiki> tags! So I went to see how you made your Winter code show up, and it appears your examples have formatting inside, which must keep Winter from taking action. --Bytesmiths 05:19, 20 December 2005 (UTC)Reply

The example code displays here on meta.wikimedia.org because Winter isn't installed. Nowiki isn't honored because Winter isn't part of the wiki rendering engine, though I can see how there is a need for a way to display Winter code on a wiki that is running Winter so i'll have to add a command --Frantik 16:19, 21 December 2005 (UTC)Reply

Fatal error

[edit]
{{#function
           | unitdictionary
          || {{#substr
                      | {{Units:{{#var|1}}}}
                      | {{#strpos
                                 | {{Units:{{#var|1}}}}
                                 | "^!dictionary=                  "
                        }}
                      | {{#strpos
                                 | {{Units:{{#var|1}}}}
                                 | "            ^!category="
                        }}
             }}
}}

returned

Fatal error: Call to undefined function: hasparamnum() in /wiki/extensions/Winter.php on line 602

Thought you'd want to know. Simetrical 06:08, 9 January 2006 (UTC)Reply

To clarify, I was trying to create a function that, given an input x, would go to the page "Units:x" (Units: is a namespace on the wiki I'm using) and return all text between
"|dictionary=                  "
and
"            |category="
(with no quote marks in the file). Simetrical 06:22, 9 January 2006 (UTC)Reply
oops, there is a bug in the strpos code. I fixed it for the next release. The syntax of your function definition has quite a few errors in it though. You probably only want two } after each var call.. the set of 4 is actualy closing the strpos function before you tell it the string.. ie: your code translates to this: (strpos (var 1)) "string") when it should be (strpost, (var 1) "string) Frantik 02:02, 12 January 2006 (UTC)Reply
actually, i see what you're trying to do.. you need to use a 'trans:' call to get the text of the wiki page. you also might want to use escape characters because mediawiki might try to trans {{#var|1}} instead of trans:(pagenane). Frantik 02:59, 12 January 2006 (UTC)Reply

Thanks. I figured I may well have been doing something wrong, but I'm glad to hear I could help you fix a bug as well. What do you mean by a trans: call? {{trans:User:Frantik}}, for instance, just gives a redlink to Template:Trans:User:Frantik, and Help:Template makes no mention of a trans: prefix. Simetrical 05:20, 12 January 2006 (UTC)Reply

You need to transclude the page. it's not done by trans: like i thought, just by prepending a : to the link (eg {{:User:Frantik}}) Frantik 09:01, 12 January 2006 (UTC)Reply
Actually, transclusion works differently depending on the namespace.. your use is probably fine, though you still may want to use escape characters [[[w:en:Wikipedia:Transclusion_costs_and_benefits|Transclusion_costs_and_benefits]]] Frantik 09:28, 12 January 2006 (UTC)Reply

Inefficiency

[edit]
How resource-efficient is Winter? I would think a string-position search should take near-zero processing time regardless of string length, but a strpos of the form
{{#strpos|{{insert a page here}}
         | word
}}
seems to take forever if I don't get a server error. I'm worried that I'll exceed my server capacity if I use too many of these commands. Is this normal? Simetrical 06:51, 9 January 2006 (UTC)Reply
Hmm I would expect it to take very little time as well. let me investigate. Frantik 01:50, 12 January 2006 (UTC)Reply
I fixed this problem as well.. the error was created when strpos returned false. Your query was returning false due to your use of whitespace; strpos was searching for word</pre> and not just word. try this: {{#strpos|{{insert a page here}} | word }} instread. Unline most languages, whitespace IS important in Winter. This is because certain text formatting will cause Mediawiki to add html to it, before Winter sees the code --Frantik 02:15, 12 January 2006 (UTC)Reply
On a more general note, I've tried to make Winter as resource efficient as possible. I also have made it a goal to never allow Winter to cause a server 500 error.. if you ever get one please let me know how you got it so i can fix it :) Frantik 02:41, 12 January 2006 (UTC)Reply

Hmm, I've gotten a number of server 500 errors. I'll be sure to make a note of them in the future and tell you.

As for the specific strpos I mentioned, would adding quotes to the string allow me to split it up across lines? It's much easier to read that way for more complicated functions. Simetrical 05:22, 12 January 2006 (UTC)Reply

since the problem is wikimedia adding formatting before winter sees it, you might try putting the code inside of a <nowiki> block. Frantik 09:05, 12 January 2006 (UTC)Reply

Problems with Variablename

[edit]

I have problems with variable some minus sign to have (example: super-param). This variables are not correctly recognized! --Thomas Klein 07:56, 6 February 2006 (UTC)Reply

Only alphanumeric characters and _ are allowed in a variable name if you want to use the variable shortcut syntax (ie {{#varname | + | 1}} as opposed to {{#var | varname | + | 1 }}
try using "super_param" instead :)
Frantik 18:16, 11 February 2006 (UTC)Reply
With _, i have the same problem. --Thomas Klein 11:43, 14 February 2006 (UTC)Reply
When I add this to a page, it displays 2
{{#setvar s_1 1}}{{#s_1 + 1}}
I'm not sure why _ isn't working on your system. --Frantik 08:07, 16 February 2006 (UTC)Reply

Simple questions

[edit]

I have some questions. Can Winter variable be a string or only number? When I write template in Winter, can i use template parametr as Winter variable?

variable names and values can be numbers or strings. You can use template parameters just you you normally would in templates: {{#substr| {{{1}}} | 1 | 2}}
--Frantik 10:41, 24 March 2006 (UTC)Reply

Arrays

[edit]

I'm trying to get array functionality out of winter. I have a template with the following code in it:

<div style="border:solid #cccccc 1px;margin:0;padding:0;line-height:0;font-size:0;display:block;">
{{#setvar|width|{{{1}}}}}
{{#setvar|height|{{{2}}}}}
{{#for | {{#setvar|i|0}} || {{#i | < | {{#height}} }} || {{#i|++}}
|| {{#for | {{#setvar|j|0}} || {{#j | < | {{#width}} }} || {{#j|++}}
|| [[Image:^_(^_(^_({{#op | {{#i}} | * | {{#height}} | + | {{#j}} | + | 3}}^_)^_)^_)Tet.png]]
}}
<br/>
}}
</div>

The line in question here is the one with the wiki Image markup. What I'm looking to do is access the template parameter number (i*height)+j+3. This is just the math needed to store a matrix in a 1d array, with an offset since params 1 and 2 are width and height.
Right now a call like this:

{{WinterPlayingField|10|1|j|j|j|j| | |j|j|j|j}}

Evaluates like this:

[[Image:{{{3}}}Tet.png]][[Image:{{{4}}}Tet.png]][[Image:{{{5}}}Tet.png]][[Image:{{{6}}}Tet.png]][[Image:{{{7}}}Tet.png]][[Image:{{{8}}}Tet.png]][[Image:{{{9}}}Tet.png]][[Image:{{{10}}}Tet.png]][[Image:{{{11}}}Tet.png]][[Image:{{{12}}}Tet.png]]

If only those {{{x}}} were parsed by the template I'd have my solution! I already tried using ^( instead of ^_(, but that causes the curly braces to be interpreted wrong. Thanks in advance for your help.

this problem is occuring because the triple brace variable replacement only occurs once before control is passed to Winter. The best way to solve this is to set the parameters to Winter variables. Unfortunately you'll have to set them all explicitly, so the code will not allow for an unlimited amount of parameters.
Try this:
{{#setvar|param1|{{{1}}}}}
{{#setvar|param2|{{{2}}}}}
{{#setvar|param3|{{{3}}}}}
{{#setvar|param4|{{{4}}}}}
{{#setvar|param5|{{{5}}}}}
{{#setvar|param6|{{{6}}}}}
{{#setvar|param7|{{{7}}}}}
{{#setvar|param8|{{{8}}}}}
{{#setvar|param9|{{{9}}}}}
{{#setvar|param10|{{{10}}}}}
{{#setvar|param11|{{{11}}}}}
{{#setvar|param12|{{{12}}}}}

<div style="border:solid #cccccc 1px;margin:0;padding:0;line-height:0;font-size:0;display:block;">
{{#setvar|width|{{{1}}}}}
{{#setvar|height|{{{2}}}}}
{{#for | {{#setvar|i|0}} || {{#i | < | {{#height}} }} || {{#i|++}}
|| {{#for | {{#setvar|j|0}} || {{#j | < | {{#width}} }} || {{#j|++}}
|| [[Image:{{#var|param{{#op | {{#i}} | * | {{#height}} | + | {{#j}} | + | 3}}}}Tet.png]]
}}
<br/>
}}
</div>
--Frantik 08:48, 11 August 2006 (UTC)Reply

Dynamic categories

[edit]

I've been using (and loving) Winter now, but came across something I just haven't been able to do. In Winter, this code:

{{#if | {{#op {{{1}}} < 100 }} | [[Category:Value 0-99]]}}

doesn't parse as I'd like it. The hope is to create categories that group items together. Their value is sent to a template and the grouping/categories are done by the template.

The categories seem to be evaluated before winter gets to them, however. In the above example, all items are put in that category.

This:

[[{{#if | {{#op {{{1}}} < 100 }} | Category:Value 0-99]]}}]]

Adds either [[]] or [[Category:Value 0-99]] to the text of the article - it is not turned into a link or category note.

I've tried some other variations (using #if to add/remove <noinclude></noinclude> statements, for instance), but haven't had luck. It just seems every way I try the category check is done before Winter is run. I was hoping there was some way or another I could fool the system, but no luck so far. Is there any way this can be done with Winter (or some other tool you know off hand, if Winter runs too late in the process)?

I'm not sure if this is possible unfortunately --Frantik 22:35, 11 August 2006 (UTC)Reply

Question on variable scoping

[edit]

I couldn't find any description of variable scoping. Can I reference a varaiable defined outside of a function inside the function? For example:

<nowinter>

{{#setvar|rate|20}}
{{#function|foo||{{#var|rate}}}}
foo={{#foo}}
rate={{#rate}}

</nowinter>

Gives me:

 foo=
 rate=20
Variables are always local, never global. If you define one outside a function, you cannot use it inside the function, and vice-verse
--Frantik 10:58, 17 December 2006 (UTC)Reply

"Call-time pass-by-reference"

[edit]

I just installed Winter 2.0.1 on MediaWiki 1.10.0 with PHP 5.2.2. I did nothing other than place Winter.php in the extensions directory and add 'require_once("extensions/Winter.php");' to the bottom of LocalSettings.php (similar to many other extensions I have installed). I started getting the error following error on all pages on my site immeditely after installing Winter:

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of call_user_func(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. in /..../extensions/Winter.php on line 1658

I'm very new to PHP but the documentation on my web host's site seems to incdicate that I can change this setting (allow_call_time_pass_reference) but am advised not to (ie to change the offending PHP instead). Did I install something wrong? Does this PHP setting normally have to be changed when using Winter? Is this indicative of some other problem? Possibly a conflict with another extension I have installed already?

Any help would be greatly appreciated!

Graham. 70.18.28.113 06:02, 12 August 2007 (UTC)Reply

Hi Graham,
I’ve solved the issue, which I missed because allow_call_time_pass_reference is enabled by default. However it is disabled in the "recommended" setup. I will release another version shortly. In the meantime you can change these lines to solve the problem:
Line 1643:
$ret = call_user_func($functionString[1],&$arr);
to
$ret = $functionString[1]($arr);

Line 1658
$ret = call_user_func($functionString[1],&$arr, $flag);
to
$ret = $functionString[1]($arr, $flag);
--Frantik 09:58, 12 August 2007 (UTC)Reply

This issue has been resolved in Winter 2.0.2 --Frantik 13:25, 12 August 2007 (UTC)Reply

Hi - I just tried the workaround and the update (2.0.2) - both appear to work just as advertized. Thanks much for the quick fix! - Graham 70.18.28.113 03:47, 13 August 2007 (UTC)Reply

Usage of Winter in MediaWiki Templates

[edit]

Hello, and thanks for a great extension! We are using mediawiki to arrange a rather large set of articles for our company, and we use templates to automatically generate categories.

The code we have in the article is as follows:

{{Article info
  | frameworks     = ISO 9000, ISO 20000
}}

and in the template "Article info", the code is as follows:

<table>
  <tr style=' vertical-align: top;'>
    <th style="text-align: right;">Frameworks:</th>
    <td>
      {{#setvar | FW | {{{frameworks}}} }}
      {{#explode | , | {{#var|FW}} | myarr }}
      {{#foreach | myarr || [[:Category:{{#_v}}]] }}
    </td>
  </tr>
</table>

This generates the following:

 Frameworks: 	[[:Category:ISO 9000]][[:Category: ISO 20000]] 

But we would like to have an actual link to the category itself, not just the text.

I understand that this has something to do with the order MW and Winter executes the parser, and I've tried using the <WinterPreWiki> tags.

Is there anything we can do to get this to work?

Thanks a lot for your time in advance, Gustavo

The problem comes from the fact that Templates are not loaded until the MediaWiki engine has parsed the page. So you can't load the contents of a template and process it before MediaWiki, and thus you can't create dynamic links using templates. You might try creating URLs with HTML instead of relying on the MediaWiki parser to create the links. --Frantik 00:40, 20 January 2009 (UTC)Reply
Some of this functionality might be possible with Winter 2.2.0 and the include functions --Frantik 10:50, 21 May 2009 (UTC)Reply

strpos returns empty string for first string

[edit]

Hi there.

This might just be the way I'm using this but consider this example:

{{#setvar|Text|hello, goodbye, good day, good night}}

{{#strpos|{{#var|Text|hello}}
{{#strpos|{{#var|Text|goodbye}}
{{#strpos|{{#var|Text|good day}}
{{#strpos|{{#var|Text|good night}} 

The 2nd, 3rd and 4th strpos return a number. However, the first strpos returns null. No doubt because coders always start at zero ;) However, I would expect it to return a zero rather than null though in this case, but it doesn't.
So, if you then want to do a #if based on the strpos result, you will never get it to evaluate on the first strpos string. Any idea how I can get around that?
I'm using Winter 2.2.0 Thanks!
--Mitchelln 17:15, 2 March 2010 (UTC)Reply

indeed.. i've fixed it in my source, but i'm not sur when I will release an update. You may wish to update your source code as well.
					case 'strpos':
						if ($param = $this->hasParamNum(3,$functionString))
						{  $wReturn = strpos($param[1],$param[2], $param[3]);
						   $wReturn  = ($wReturn !== false)?$wReturn:'';
						}
						else if ($param = $this->hasParamNum(2,$functionString))
						{  $wReturn = strpos($param[1],$param[2]);
						   $wReturn  = ($wReturn !== false)?$wReturn:'';

is the new source code..

btw you could rewrite your code like this

{{#Text| @= |hello, goodbye, good day, good night}}

{{#strpos|{{#Text}}|hello}}
{{#strpos|{{#Text}}|goodbye}}
{{#strpos|{{#Text}}|good day}}
{{#strpos|{{#Text}}|good night}} 
--Frantik 15:52, 3 March 2010 (UTC)Reply

Oh, fantastic stuff! Thanks for the quick fix. Saves me a lot of string concat bother to get round the problem
Thanks for the usage tip as well, much neater than mine :)

--Mitchelln 13:42, 5 March 2010 (UTC)Reply
Hmm, just tried the fix above and it still does not work, the following #if does not evaluate true on the first string :(
{{#if|{{#strpos|{{#Text}}|hello}}}}

Any ideas?
Thanks for your help with this

--Mitchelln 13:49, 5 March 2010 (UTC)Reply
0 still evaluates as false, so you need to check that strpos is not an empty string
{{#ThePos | @= | {{#strpos| {{#Text}} | hello}} }}
{{#if| {{ {{#ThePos}} | != | "" }} || ....
I think that should do it --Frantik 18:15, 6 March 2010 (UTC)Reply


Hi. Thanks for getting back to me. Sorry to be a pain, but I just can't get this to work properly. I have:
{{#ThePos | @= | {{#strpos| {{{Icons Codes}}} | Wheelchair Access}} }}
{{#if| {{{{#ThePos}} | != | "" }} || {{#Icon:Wheelchair Access.png|Wheelchair Access||Wheelchair Access}}}}


and what I get is:
!=|| <the icon> }}

I've tried everything I can think of, I think I must be going crazy! Sorry to be a pain.

--Mitchelln 9:03, 8 March 2010 (UTC)
my bad.. there's a syntax error..
{{#if| {{{{#ThePos}} | != | "" }} || ...
should be either
{{#if| {{#ThePos | != | "" }} || ...
or
{{#if| {{#op |{{#ThePos}} | != | "" }} || ...
also, i'm not sure if there should be a double pipe in your #Icon code..
--Frantik 09:46, 20 March 2010 (UTC)Reply
Thanks for that. Got it working! The double pipe in the #icon is the correct syntax for that extension. I use the link= functionality of MW image handling now though - saves an extension!.
--Mitchelln 14:03, 21st April 2010 (UTC)

#If statement displays rubbish if bool is a URL

[edit]

Hi. I've got the following which shows external links if any are present:

{{#if {{{Links|}}}||
|-valign="top"
! External Links
| {{{Links|}}}
}}

Instead of the URL showing, for example:
http://www.example.com

What I see is:
class="external free" title="http://www.example.com

I'm using Winter 2.2.0 and MW 1.15.1. Any ideas what might be going on?

Thanks!
Mitchelln 16:43, 28th April 2010 (UTC)

The first thing I notice is the syntax is wrong
{{#if| {{{Links|}}}||
|-valign="top"
! External Links
| {{{Links|}}}
}}
not sure if that will fix your problem though --Frantik 23:15, 7 May 2010 (UTC)Reply
Hi there. Thanks for your reply.
Sorry, missing | was a typo in my post. Still doesn't work I'm afraid :(
Mitchelln 09:33, 17th May 2010 (UTC)
What is the value of Links? I tried using standard wiki text and it worked fine.. these all work as expected:
{{#if| [[Main Page]] || [[Main Page]] }}
{{#if| http://www.example.com/ || http://www.example.com/ }}
{{#if| [http://www.example.com/] || [http://www.example.com/] }}
Make sure you read this section of the manual for more info.. it's probably some weird problem with Mediawiki processing the HTML.. class="external" is part of the HTML for a link --Frantik 12:00, 18 May 2010 (UTC)Reply

Operating on spaces

[edit]

How do I pass explicit space (ascii 32) characters to #strpos and other function? (Winter 2.2.0 on MW 1.15.0)

  • Position of 'W' in 'Robin Williams': {{#strpos|Robin Williams|W}} returns 6
  • Position of ' ' in 'Robin Williams': {{#strpos|Robin Williams| }} returns (empty) not 5

Similarly, {{#explode| |Robin Williams}} does not split at the space delimiter. These functions seem to think nothing is being passed - possibly trimming the input?

How do I force searching for a space in #strpos? The "escape" characters do not list a way to escape a space character.

Oddly, {{#strpos|Robin Williams|R}} returns (empty), not 0. I suppose false is evaluated as 0 when used in a numeric context, but 0 should be returned so one can test if the string is found (0) vs not found (empty).

--DavidBiesack 14:11, 21 September 2010 (UTC)Reply

To define a space you need to use quotes. {{#strpos|Robin Williams| " " }}
I'm not sure if i fixed strpos since the last version but it's returning 0 on my wiki --Frantik 04:47, 24 September 2010 (UTC)Reply

Bug with #if

[edit]

With this expression:

{{#if | {{#strpos|Robin Williams| " " }}  || First Name: "{{#substr|Robin Williams|0|{{#strpos|Robin Williams|" "}}}}" Last Name: "{{#substr|Robin Williams|{{#op | {{#strpos|Robin Williams|" "}} + 1}} }}" || ''Name does not contain spaces'' }}

I expect to get:

First Name: "Robin" Last Name: "Williams"

but on the Winter Sandbox running 2.2.1 the result is missing the last quote:

First Name: "Robin" Last Name: "Williams

On my install running 2.2.0 I get an error:

First name: "# ERROR (3): Syntax error in (substr|Robin Williams|0|" Last Name: "Williams" || Name does not contain spaces).

Will 2.2.1 be available for download soon?

--DavidBiesack 15:37, 24 September 2010 (UTC)Reply

it's not a bug.. the trailing " is interpreted differently.. see this section of the manual
I still consider this a bug. The parameter does not begin with a ", so the last " should not be interpreted as a string close character. (Perhaps if you recognize ^" as an escaped quote, then I could use that at the end instead of a bare " ?)--DavidBiesack 00:14, 26 September 2010 (UTC)Reply
It's just a quirk of the language, not a bug --Frantik 00:57, 6 September 2011 (UTC)Reply
as for when 2.2.1 will be out, i need to check it for security holes for some of the newer functions i added.. but i guess there are some other bug fixes i did like this one so perhaps ill try to do it soon --Frantik 02:01, 25 September 2010 (UTC)Reply
thanks. Even a bug fix release would help me integrate this. currently we only have Winter in our test environment; we can't put it into production until this above bug (the ERROR (3): Syntax error" bug above) is fixed/released. --DavidBiesack 00:14, 26 September 2010 (UTC)Reply
2.2.1 released  :) --Frantik 03:59, 27 September 2010 (UTC)Reply

Nested #if issues

[edit]
{{#if | 1 || level one {{#if | 1 || level two {{#if | 1 || level three}} }} }}

should yield "level one level two level three", but instead it yields "level one {{#if | 1 ". It looks like the double-pipe from the second #if is being interpreted as the "if false" flag for the first #if. But if you only have two nested #ifs, then it comes out right. --MidnightLightning 13:33, 25 October 2010 (UTC)Reply

thanks for pointing this out.. i'll look into it when i get a chance --Frantik 23:05, 26 October 2010 (UTC)Reply
this problem has been solved and will be fixed in the next release.. nesting in general just got a whole lot better --Frantik 01:23, 4 November 2010 (UTC)Reply

Security and Winter

[edit]

Winter allows you to edit the generated HTML, which potentially could allow someone to insert javascript or other malicious html. I'm looking into ways to use the Mediawiki sanitizer to clean the output, but for now it should not be enabled on sites where untrusted users can edit the code. --Frantik 10:09, 4 November 2010 (UTC)Reply

I think one way to avoid security issues is to make Winter default to "WinterPreWiki" status, and then add a "WinterPostWiki" tag which will work like Winter does currently. Then i will make "WinterPostWiki" optional. This will ensure that public sites could use Winter without worry of users being able to create arbitrary html/javascript code. It will cause backwards compatibility problems though unfortunately. Any comments on this idea? --Frantik 04:05, 5 November 2010 (UTC)Reply
Did you try to use the InternalParseBeforeLinks hook instead of ParserBeforeTidy with using Sanitizer::removeHTMLtags() afterwards? Looking at the internal parser processes and debugging step by step can reveal the right place (hook) to do certain things, perhaps there is another hook before the sanitizing is going on. I am still missing some kind of InternalParseBeforeSanitize though. --Danwe 02:11, 24 December 2011 (UTC)Reply

<winterprewiki> is not working

[edit]

I tried to switch the categories using the <winterprewiki> tag. It did absolutly not work. Maybe I´m using it in the wrong way?

I made a testpage:

A tag for every command (I also tried only one tag, same effect) http://www.bluewater-info.net/wiki_/Aberdeen_Hotel

Output: <winterprewiki></winterprewiki> <winterprewiki></winterprewiki> <winterprewiki></winterprewiki> <winterprewiki></winterprewiki> Categories: POI | POI de

The whole content of the page is the POI_common template:

<includeonly><winterprewiki>{{#var | Language | @= | _{{SUBPAGENAME}} }}</winterprewiki>
<winterprewiki>{{#var | Language | @= | {{#if | {{#strpos | {{PAGENAME}} | / }} || {{#var | Language }} || _en }} }}</winterprewiki>
<winterprewiki>{{#ifeq | {{{Language}}} | _en | [[Category:POI]] }}</winterprewiki>
<winterprewiki>{{#ifeq | {{{Language}}} | _de | [[Category:POI_de]] }}</winterprewiki></includeonly>

The template should just descide if it´s a german or an english page. The variable {{{Language}}} is "_en", so category should be just POI and not POI_de

A check on the Version page shows the following tags:

<anyweb>, <coordinates>, <display map>, <display point>, <display points>, <distance>, <finddestination>, <gallery>, <geocode>, <geodistance>, <html>, <htmlet>, <languageselector>, <listerrors>, <math>, <nowiki> und <pre>

"nowinter" and "winterprewiki" are not in the list?

Winter Version is 2.2.1 on Mediawiki 1.16, SMW 1.5.4

Best regards, --Tkrueger73 04:43, 28 December 2010 (UTC)Reply

Winterprewiki does not work in templates. Templates are not transcluded until AFTER winterprewiki is processed. (Thats the idea behind winterprewiki, is to work before the wiki starts processing stuff). Try using #include instead of using a template --Frantik 00:55, 6 September 2011 (UTC)Reply

$wgWinterNamespaces

[edit]

using the example i have added this line to my Localsttings.php: $wgWinterNamespaces = array( 2 => false, 3 => false); and winter is still allowing me winter function and have it run on a user talk page. what am i doing wrong

MediaWiki 1.16.2 PHP 5.3.5 (apache2handler) MySQL 5.5.8

Winter (Wiki INTERpreter) (Version 2.2.1)

Suberglobal variables bug (same as in Extension:Variables, fixed in v1.3 and v2.0)

[edit]

I believe (from reviewing the code and my own experience) that this extension has the same bug as old versions of Extension:Variables which I have fixed in version 1.3 and in 2.0 for good. This was an very random and hard to track issue where the extension accidentally enabled some kind of super-global variables behavior where variable values would be transmitted between pages. This could happen for example during Special:Import imports, some Semantic MediaWiki maintenance script for refreshing data, perhaps even during common job-queue handling. Basically this happens when the wiki is handling several page renderings in one session. First I solved this in v1.3 by using the ParserClearState hook but this raised another bug (like special page inclusion would reset all variables), fixed in Variables 2.0 by having one variable store per Parser instance rather than one global one but still using ParserClearState so all variables are rest after one call to Parser::parse() has finished. --Danwe 01:48, 24 December 2011 (UTC)Reply