[Bug 30513] Redirect tag is not resolved in XML dump file

2012-09-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Diederik van Liere dvanli...@gmail.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

-- 
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 30513] Redirect tag is not resolved in XML dump file

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

--- Comment #21 from Diederik van Liere dvanli...@gmail.com 2012-01-24 
01:22:51 UTC ---
The current solution (as in trunk) is fine: it leaves the title as is and adds
a new namespace tag. This tiny bit of redundancy (having the namespace as an
integer in a separate tag and the fully qualified namespace in the title) will
make it much more easier to select articles form a specific namespace. In
particular, with the added gender specific namespaces it becomes only harder to
parse out the namespace that an article belongs to. So the current is that
there are no incompatible changes, just for each article an extra namespace
tag.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #19 from Brion Vibber br...@wikimedia.org 2011-11-22 19:00:29 UTC 
---
Marking the namespace id number is possibly not quite sufficient without
specifying the prefix up in the namespaces section. Probably enough for search
purposes, but consider adding aliases up in the namespaces section in
siteinfo as well.

To resolve the Lucene search indexer's issue with this (bug 32376) which causes
the search prefix problem (bug 31697), the Lucene indexer will also need to be
updated to *use* the field.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #20 from Brion Vibber br...@wikimedia.org 2011-11-22 19:21:24 UTC 
---
Per IRC disucssion with Ariel a simpler one-off fix for the search stuff is:
* per (bug 32376) switch to using the canonical names in XML export (so
'Usuario:' not 'Usuaria:'); the MWSearch prefix search implementation will
rewrite the titles to gendered form in the actual search results.

Looking over the patch:
+Version 0.6 adds a separate namespace tag, locates the
+redirect target and strips of the namespace from the title. 

it doesn't actually appear to be stripping the namespace from the title, it's
still using getPrefixedText()... and if it did strip it that would be a serious
breaking change in the format.

Can you clarify what this is meant for?

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Brion Vibber br...@wikimedia.org changed:

   What|Removed |Added

 Blocks|32376   |

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Diederik van Liere dvanli...@gmail.com changed:

   What|Removed |Added

 Blocks||32376

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #18 from Diederik van Liere dvanli...@gmail.com 2011-11-10 
18:07:10 UTC ---
Created attachment 9414
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=9414
Fixes issues as suggested by Platonides

The previous patch is still needed and is not obsolete.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #11 from Diederik van Liere dvanli...@gmail.com 2011-11-09 
18:44:09 UTC ---
Created attachment 9400
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=9400
New patch adds export-0.6.xsd and fixes minor != null check

Based on IRC feedback, made changes, including adding a new export-0.6.xsd
file.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #12 from Platonides platoni...@gmail.com 2011-11-09 18:47:28 UTC 
---
Note for the committer: export-0.6.xsd should be added with export-0.5.xsd as
history.

Stylistic issue: Add spaces around $redirect !== null
You're still using strval($redirect) (Brion point #4)

As I said on irc, I'm not convinced about adding the new ns tag, as that
information is already available.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #13 from Diederik van Liere dvanli...@gmail.com 2011-11-09 
19:12:04 UTC ---
Created attachment 9401
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=9401
Updated history section of export-0.5.xsd, export-0.6.xsd and added explicit
string conversion to title object.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Reedy s...@reedyboy.net changed:

   What|Removed |Added

   Attachment #8973|0   |1
   is patch||
   Attachment #8973|application/octet-stream|text/plain
  mime type||

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Reedy s...@reedyboy.net changed:

   What|Removed |Added

   Attachment #8973|0   |1
is obsolete||

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Aaron Schulz aschulz4...@gmail.com changed:

   What|Removed |Added

 CC||aschulz4...@gmail.com

--- Comment #14 from Aaron Schulz aschulz4...@gmail.com 2011-11-09 20:39:29 
UTC ---
The old patches should be marked obsolete.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #15 from Aaron Schulz aschulz4...@gmail.com 2011-11-09 20:40:50 
UTC ---
Overall patch seems sane (it will still need and get more code review once
committed).

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Diederik van Liere dvanli...@gmail.com changed:

   What|Removed |Added

   Attachment #8962|0   |1
is obsolete||

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Diederik van Liere dvanli...@gmail.com changed:

   What|Removed |Added

   Attachment #9400|0   |1
is obsolete||

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Diederik van Liere dvanli...@gmail.com changed:

   What|Removed |Added

   Attachment #9394|0   |1
is obsolete||

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #16 from Platonides platoni...@gmail.com 2011-11-09 23:07:53 UTC 
---
(In reply to comment #13)
 Created attachment 9401 [details]
 and added explicit string conversion to title object.

No. We didn't mean you should change $title-getPrefixedText() to strval(
$title-getPrefixedText() ) ), which is redundant.
But to use $redirect-getPrefixedText() instead of strval( $redirect )

The if condition could be changed to $redirect instanceOf Title 
$redirect-isValidRedirectTarget() to make sure it's appropiate (just copied
the codnition from Title.php:396).

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #17 from Aaron Schulz aschulz4...@gmail.com 2011-11-09 23:20:35 
UTC ---
(In reply to comment #16)
 The if condition could be changed to $redirect instanceOf Title 
 $redirect-isValidRedirectTarget() to make sure it's appropiate (just copied
 the codnition from Title.php:396).

Yep, that would also make using getPrefixedText() OK since getRedirectTarget()
can return a string and not a Title.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #10 from Diederik van Liere dvanli...@gmail.com 2011-11-09 
02:27:18 UTC ---
Created attachment 9394
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=9394
Patch adds a new namespace tag and determines redirect target.

As requested by Sumana, updated to most recent version of trunk. Patch also
includes version increment.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-11-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #9 from Sumana Harihareswara suma...@panix.com 2011-11-02 
23:24:49 UTC ---
Diederik, could you re-upload your updated patch and ensure it's in the right
format (and that it applies cleanly to trunk as is)?  Right now, attachment #
8973 is in a format such that Bugzilla won't show the diff automatically...

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-09-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

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

   What|Removed |Added

   Keywords||need-review, patch
 CC||suma...@panix.com

--- Comment #8 from Sumana Harihareswara suma...@panix.com 2011-09-30 
16:00:43 UTC ---
Added the patch and need-review keywords; Mark hopes to get someone to
review the patch soon.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-08-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Mark A. Hershberger m...@everybody.org changed:

   What|Removed |Added

   Priority|Unprioritized   |Normal
 CC||m...@everybody.org,
   ||tpars...@wikimedia.org

--- Comment #4 from Mark A. Hershberger m...@everybody.org 2011-08-25 
15:54:23 UTC ---
Diederik, Trevor,

Could you update the patch with respect to Brion's review in comment #3 so that
it can be applied?

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-08-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #5 from Diederik van Liere dvanli...@gmail.com 2011-08-26 
00:34:09 UTC ---
Created attachment 8973
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=8973
Updated patch

Updated patch; fixes issues as mentioned by Brion.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-08-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #6 from Diederik van Liere dvanli...@gmail.com 2011-08-26 
00:35:03 UTC ---
New patch fixes issues 1,3 and 4. 
Brion: could you elaborate on issue 2 and possible ways to handle that? I am
not sure what to do about 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 30513] Redirect tag is not resolved in XML dump file

2011-08-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #7 from Diederik van Liere dvanli...@gmail.com 2011-08-26 
00:41:21 UTC ---
And this patch also included a separate namespace tag, sorry for not
mentioning.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-08-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #3 from Brion Vibber br...@wikimedia.org 2011-08-23 18:09:25 UTC 
---
Some issues:

* no matching update to XML schema version  definition
* redirect lookup _may_ not be fully synchronized with the page data snapshot
* what does this do in the null case? No field or empty field? If this is an
implicit choice, it should probably be made more explicit.
* relies on implicit string conversion of Title object which... appears to give
text formatting by default. Wouldn't hurt to make this explicit.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-08-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

--- Comment #1 from Diederik van Liere dvanli...@gmail.com 2011-08-22 
20:40:59 UTC ---
Created attachment 8962
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=8962
This patch resolves the page_is_redirect field in the xml dump file. It will
add an attribute name to the redirect tag so it should be backwards
compatible.

A huuuge thanks to Trevor who basically wrote this patch so all credits go
to him if this works and all complaints should go to me in case this is a bad
idea.

-- 
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 30513] Redirect tag is not resolved in XML dump file

2011-08-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=30513

Roan Kattouw roan.katt...@gmail.com changed:

   What|Removed |Added

 CC||roan.katt...@gmail.com

--- Comment #2 from Roan Kattouw roan.katt...@gmail.com 2011-08-22 22:02:20 
UTC ---
Patch looks good to 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