[Bug 33420] Validator extension: A new page type.

2012-03-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

Van de Bugger van.de.bug...@gmail.com changed:

   What|Removed |Added

 Status|RESOLVED|VERIFIED

--- Comment #14 from Van de Bugger van.de.bug...@gmail.com 2012-03-01 
20:47:16 UTC ---
Verified on r112454.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #13 from Van de Bugger van.de.bug...@gmail.com 2012-02-29 
17:50:00 UTC ---
As I said, I wrote a front-end for Validator to use in templates. So, I have
to have string representation for all the types. I decided not to introduce
more entities but use values of TYPE_xxx constants. So, title type is
represented in PHP code as TYPE_TITLE, and as title in template. See example
above.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

Daniel Werner danwe...@web.de changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 CC||danwe...@web.de
 Resolution||FIXED

--- Comment #11 from Daniel Werner danwe...@web.de 2012-02-28 21:53:49 UTC ---
The only reason I originally used a capital letter on this was that in PHP
basic primitive types such as bool, int or string use lower-case while for
object-types we usually use upper case.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #12 from Jeroen De Dauw jeroen_ded...@yahoo.com 2012-02-28 
22:04:17 UTC ---
I figured as much. Still, these values are not meant to reflect whatever the
name of the corresponding type in PHP is.

Although I'd go for keeping it all lower case, I'm wondering what the
difference is. Van, are you comparing to the string value for some reason? You
should use the constants...

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #9 from Van de Bugger van.de.bug...@gmail.com 2012-02-27 20:12:03 
UTC ---
Let me check it with my code.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #10 from Van de Bugger van.de.bug...@gmail.com 2012-02-27 
21:55:03 UTC ---
Ok, this works for me.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #7 from Van de Bugger van.de.bug...@gmail.com 2012-02-26 22:20:30 
UTC ---
Ping. Please convert `Title' type name to lower case.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #8 from Jeroen De Dauw jeroen_ded...@yahoo.com 2012-02-26 
23:08:02 UTC ---
r112454

Can this be closed, or do you still need me to do something?

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33420] Validator extension: A new page type.

2012-02-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

Sumana Harihareswara suma...@panix.com changed:

   What|Removed |Added

 CC||jeroen_ded...@yahoo.com,
   ||suma...@panix.com
Summary|Validator: A new `page' |Validator extension: A new
   |type.   |page type.

--- Comment #2 from Sumana Harihareswara suma...@panix.com 2012-02-14 
12:39:48 UTC ---
Adding Jeroen to cc list since he maintains the Validator extension.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
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 33420] Validator extension: A new page type.

2012-02-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

Jeroen De Dauw jeroen_ded...@yahoo.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #3 from Jeroen De Dauw jeroen_ded...@yahoo.com 2012-02-14 
14:17:26 UTC ---
AFAIK this has already been applied by Van himself.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
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 33420] Validator extension: A new page type.

2012-02-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

Van de Bugger van.de.bug...@gmail.com changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |

--- Comment #4 from Van de Bugger van.de.bug...@gmail.com 2012-02-14 17:09:28 
UTC ---
 AFAIK this has already been applied by Van himself.

No. It was not me. Daniel Werner added something very similar, but named it
Title, not Page. I do not care about this change, but:

Why new type name is capitalized? Other type names are of lower case: `string',
`float', `boolean', but this one is `Title'. I want it in lowercase, like other
types, because these values are user-visible in my another extension.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
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 33420] Validator extension: A new page type.

2012-02-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #5 from Jeroen De Dauw jeroen_ded...@yahoo.com 2012-02-14 
17:21:20 UTC ---
 It was not me. Daniel Werner added something very similar

Oops, sorry. I got confused there :)

 Why new type name is capitalized?

It should not be. I will fix this in a moment.

 in my another extension.

Which one?

So if the capitalization issue is fixed, does the title addition help you out
or not? If not, please create a new patch that includes the new Criterion and
Manipulation classes which you apparently forgot to add.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
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 33420] Validator extension: A new page type.

2012-02-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

Jeroen De Dauw jeroen_ded...@yahoo.com changed:

   What|Removed |Added

  Component|[other] |Validator

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
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 33420] Validator extension: A new page type.

2012-02-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

Jeroen De Dauw jeroen_ded...@yahoo.com changed:

   What|Removed |Added

 AssignedTo|wikibugs-l@lists.wikimedia. |jeroen_ded...@yahoo.com
   |org |

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
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 33420] Validator extension: A new page type.

2012-02-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33420

--- Comment #6 from Van de Bugger van.de.bug...@gmail.com 2012-02-14 19:38:41 
UTC ---
 Which one?

It is not yet published. I once mentioned it -- a parser function which
provides templates with Validator functionality. For example:

 {{  #validate: 
 |   target  = type: page;   alias: 1;
 |   text= type: string; alias: 2;  default: {{ PAGENAME: {{{ 
 target }}} }};
 |   tooltip = type: string; alias: 3;  default: {{ FULLPAGENAME: {{{ 
 target }}} }};
 |   link?   = type: boolean;   default: yes;
 }}

After it, in template body {{{ target }}}, {{{ text }}}, {{{ tooltip }}}, {{{
link? }}} are ready to use.

Before it was bloacked because it required patched Validator (it uses page
type), now I have to fix it a bit -- replace `page' with `title'.

And I am not settled down with names yet...

 So if the capitalization issue is fixed, does the title addition help you out
 or not? If not, please create a new patch that includes the new Criterion and
 Manipulation classes which you apparently forgot to add.

`Title' type is the almost the same as my `Page' type, only names are differ. I
think I can use it. If capitalization is fixed, I believe I can use it.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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