[Bug 26480] add a pppageprops param to prop=pageprops

2011-01-05 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26480

Reedy s...@reedyboy.net changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #7 from Reedy s...@reedyboy.net 2011-01-05 22:12:12 UTC ---
r79675

-- 
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 26480] add a pppageprops param to prop=pageprops

2011-01-04 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26480

--- Comment #6 from Reedy s...@reedyboy.net 2011-01-05 03:20:02 UTC ---
-- Name/value pairs indexed by page_id
CREATE TABLE /*_*/page_props (
  pp_page int NOT NULL,
  pp_propname varbinary(60) NOT NULL,
  pp_value blob NOT NULL
) /*$wgDBTableOptions*/\G

CREATE UNIQUE INDEX /*i*/pp_page_propname ON /*_*/page_props
(pp_page,pp_propname)\G



Having already filtered by a page... Filtering by a propname afterwards,
wouldn't to too expensive (thinking about if we left it as an unvalidated input
string)

-- 
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 26480] add a pppageprops param to prop=pageprops

2010-12-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26480

--- Comment #5 from Bryan Tong Minh bryan.tongm...@gmail.com 2010-12-30 
10:11:50 UTC ---
Indeed

-- 
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 26480] add a pppageprops param to prop=pageprops

2010-12-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26480

--- Comment #1 from Bryan Tong Minh bryan.tongm...@gmail.com 2010-12-29 
20:38:41 UTC ---
We would need to track somewhere which pageprops exist, which I don't really
want to do. Or make it an unvalidated string param.

-- 
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 26480] add a pppageprops param to prop=pageprops

2010-12-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26480

--- Comment #2 from duplicate...@googlemail.com 2010-12-29 20:53:23 UTC ---
In my opinion a unvalidated string param is enough.

-- 
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 26480] add a pppageprops param to prop=pageprops

2010-12-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26480

--- Comment #3 from duplicate...@googlemail.com 2010-12-29 21:14:40 UTC ---
Maybe Useful for checking whether a certain page *has* a certain page prop.
is better, as description. Because a page has props and not 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


[Bug 26480] add a pppageprops param to prop=pageprops

2010-12-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26480

--- Comment #4 from Reedy s...@reedyboy.net 2010-12-30 01:00:01 UTC ---
(In reply to comment #1)
 We would need to track somewhere which pageprops exist, which I don't really
 want to do. Or make it an unvalidated string param.

IIRC, didn't Soxred93 do something like that originally, which you reverted
that part and cleaned up

(Don't take that as me disagreeing with what you did though! :))

-- 
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