Last modified: 2013-08-20 20:09:19 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T54638, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52638 - Selser regression: pages with templates that generate tables and foster content out of the table are broken
Selser regression: pages with templates that generate tables and foster conte...
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
serializer (Other open bugs)
unspecified
All All
: High normal
: ---
Assigned To: ssastry
:
Depends on:
Blocks: 52488
  Show dependency treegraph
 
Reported: 2013-08-08 09:22 UTC by Roan Kattouw
Modified: 2013-08-20 20:09 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Roan Kattouw 2013-08-08 09:22:04 UTC
At http://en.wikipedia.org/wiki/Raven-Symon%C3%A9?oldid=566906720 , if you edit anything in the document, the infobox is deleted, but if you make a null edit, the infobox is retained.

This appears to be a bug in selser: if the HTML doesn't change, the wikitext doesn't change, but any small change in the HTML results in the infobox being deleted. I verified that VisualEditor isn't dirtying anything, but just to be sure I also reproduced this on the command line by just making a 1-character change to the HTML as shown below.

I think this may have something to do with the double }} closing in the infobox.

$ mkdir tmp
$ cd tmp
$ wget 'http://parsoid.wmflabs.org/en/Raven-Symon%C3%A9?oldid=566906720' -O originalHTML
$ wget 'http://en.wikipedia.org/wiki/Raven-Symon%C3%A9?oldid=566906720&action=raw' -O originalWikitext
$ cp originalHTML editedHTML
$ vi editedHTML   # Change '<h2>Life and career</h2>' to '<h2>Life and careeer</h2>'
$ node parse.js --html2wt --selser --oldtextfile originalWikitext --oldhtmlfile originalHTML --inputfile editedHTML > newWikitext
$ diff -u originalWikitext newWikitext

--- /home/catrope/tmp/originalWikitext	2013-08-08 14:34:46.463998552 +0800
+++ /home/catrope/tmp/newWikitext	2013-08-08 17:04:41.596239434 +0800
@@ -1,29 +1,7 @@
 {{Use mdy dates|date=June 2013}}
 {{pp-move-indef}}
-{{Infobox person
-| image           = Raven-Symoné 2011.jpg
-| caption         = Raven-Symoné in 2011 
-| birth_name      = Raven-Symoné Christina Pearman 
-| alias           = Raven<BR>Raven-Symone
-| background      = solo_singer
-| instrument      = [[Singing|Vocals]], [[piano]]
-| birth_date      = {{birth date and age|mf=yes|1985|12|10}}
-| birth_place     = [[Atlanta]], [[Georgia (U.S. state)|Georgia]], U.S.<ref>{{cite web |url=http://movies.msn.com/celebrities/celebrity-biography/raven-symone/|title=Raven Symone:Biography on MSN |accessdate=2008-07-15 |year=2008 |publisher=[[MSN]]}}</ref> 
-| genre           = 
-| occupation      = Actress, singer, [[comedienne]], dancer, television producer, [[fashion model]]
-| years_active    = 1989–present
-| label           = [[MCA Records|MCA]], Crash, RayBlaze, [[Hollywood Records|Hollywood]]
-| associated_acts = [[The Cheetah Girls (band)|The Cheetah Girls]]
-| website         = [http://www.myspace.com/ravensymone Raven-Symoné Myspace page]
-| module         = {{Infobox musical artist|embed=yes
-| background          = solo_singer
-| genre               = [[Contemporary R&B|R&B]], [[pop music|pop]], [[Hip hop music|hip hop]], [[Soul music|soul]], [[dance music|dance]]
-| instrument          = Vocals, [[piano]]
-| years_active        = 1993–present
-}}
 
-}}
 '''Raven-Symoné Christina Pearman'''<ref>[http://www.mlive.com/entertainment/saginaw/index.ssf/2008/08/ravensymone_steps_out_of_chara.html]</ref>, (born December 10, 1985), known professionally as '''Raven-Symoné''' (pronounced {{IPA|/ˈreɪ.vən sɪˈmoʊn/}}, as though unaccented), or simply '''Raven''', is an American actress and singer. Raven-Symoné launched her career in 1989 after appearing in ''[[The Cosby Show]]''<!-- Per http://www.imdb.com/name/nm0712368/filmoseries#tt0086687 her first episode of "A Different World" was Nov. 2, 1989, while her first Cosby Show episode was Sept 21, 1989. --> as Olivia. She released her debut album, ''[[Here's to New Dreams]]'' in 1993; the single, "[[That's What Little Girls Are Made Of]]" charted number 68 on the US ''Billboard'' [[Hot 100]].<ref>[http://www.billboard.com/song/raven-symone/that-s-what-little-girls-are-made-of/450772#/song/raven-symone/that-s-what-little-girls-are-made-of/450772 That's What Little Girls Are Made Of – Raven-Symoné]. Billboard.com. Retrieved 2012-05-19.</ref> The next album, ''[[Undeniable (Raven-Symoné album)|Undeniable]]'', was released on May 4, 1999.
 
 Raven-Symoné appeared in several successful television series, such as ''[[The Cosby Show]]'' and ''[[Hangin' with Mr. Cooper]]'', in the late 1980s and early 1990s. From 2003 to 2007, she starred in the [[Disney Channel]] series, ''[[That's So Raven]]'' in which she played Raven Baxter, a psychic teenager who tried her best to keep her psychic powers a secret. During her time on ''That's So Raven'', Raven-Symoné released her third studio album, ''[[This Is My Time (Raven-Symoné album)|This is My Time]]'' (2004) which was her best selling solo album to date, charting at number 51 on the ''Billboard'' 200.<ref name="billboard.com">[{{BillboardURLbyName|artist=raven-symoné|chart=Billboard 200}} Raven-Symoné]. Billboard.com. Retrieved 2012-05-19.</ref> After a year of the end of ''That's So Raven'', she released her fourth studio album, ''[[Raven-Symoné (album)|Raven-Symoné]]'' (2008). The album peaked at number 159 on the ''Billboard'' 200. During 2003 to 2006, she participated in four soundtracks from Disney, [[RIAA certification|RIAA-certified]] double-platinum album, ''[[The Cheetah Girls (soundtrack)|The Cheetah Girls]]'' (2003), RIAA-certified gold album, ''[[That's So Raven (soundtrack)|That's So Raven]]'' (2004), ''[[That's So Raven Too!]]'' (2006) and RIAA-certified platinum album, ''[[The Cheetah Girls 2 (soundtrack)|The Cheetah Girls 2]]'' (2006). The soundtracks sold a combined 4.1 million copies in the U.S. alone. As of April 2008, Raven-Symoné has sold 314,000 albums in the United States.
@@ -32,7 +10,7 @@
 
 In 2012, Raven-Symoné ranked number eight on ''[[VH1]]''{{'s}} "100 Greatest Kid Stars Of All Time" list,<ref name="GKSAT">[http://blog.vh1.com/2012-12-02/and-the-1-greatest-kid-star-of-all-time-is/ Greatest Kid Star Of All Time]</ref> and ranked number one on ''Loop21''{{'s}} "10 Richest Black Actresses Under 40" list.<ref name="RBAU40">[http://www.loop21.com/entertainment/10-richest-black-actresses-under-40 10 Richest Black Actresses Under 40]</ref>
 
-==Life and career==
+==Life and careeer==
 
 ===1985–99: Early life and career beginnings with ''The Cosby Show''===
 Raven-Symoné was born in Atlanta, Georgia to Lydia (née Gaulden) and Christopher B. Pearman. Raven-Symoné is of [[African-American]] <ref>http://www.tv.com/people/raven-symone/</ref><ref>http://xfinity.comcast.net/slideshow/entertainment-aachildactors/3/</ref><ref>http://ethnicelebs.com/raven-symone</ref><ref name="imdb">http://www.imdb.com/name/nm0712368/bio</ref><ref>http://www.nndb.com/people/017/000103705/</ref> and [[Native Americans in the United States|Native American]] ancestry.<ref name="imdb" /> At age three, her family moved to [[Ossining (village), New York|Ossining, New York]] where she attended Park School.<ref name="birthplace">Charlotte Moore (January 25, 2004). [http://www.pe.com/lifestyles/teen/stories/PE_Fea_Teen_raven0125.a11bc.html Raven takes flight]{{dead link|date=September 2011}} PE.com.</ref><ref>[http://ftp.rootsweb.com/pub/usgenweb/la/winn/bios/sym1569.txt Rootsweb.com]{{Dead link|date=May 2010}}</ref> As an infant, she worked for Atlanta's Young Faces Inc. Modeling Agency and was featured in local print advertisements. At age two, she worked with Ford Models in New York City and appeared in ads for [[Ritz cracker]]s, [[gelatin dessert|Jell-O]], [[Fisher-Price]], and [[Cool Whip]].
Comment 1 ssastry 2013-08-08 15:00:12 UTC
Confirmed. 

It is not a double-closed infobox.  It is an infobox within an infobox.  Although, I have gone through a number of these scenarios of infoboxes within infoboxes earlier and handled them in our regular serializer ... and confirmed that normal WTS (wt serializer) can handle it.  Now examining what gives selser the fits.

Thanks for the thorough testing and a way to reproduce it.  Will find a reduced test case and proceed.
Comment 2 ssastry 2013-08-08 15:42:05 UTC
Reduced test case on which the problem can be reproduced:

{|
{{echo|a}}
|-
|x
}}

This is a selser regression introduced by a fix for bug 51217.

A fix will be in shortly.
Comment 3 Gerrit Notification Bot 2013-08-08 16:06:47 UTC
Change 78242 had a related patch set uploaded by Subramanya Sastry:
(Bug 52638) Fix selser regression introduced by fix for bug 51217

https://gerrit.wikimedia.org/r/78242
Comment 4 Gerrit Notification Bot 2013-08-08 16:58:57 UTC
Change 78242 merged by jenkins-bot:
(Bug 52638) Fix selser regression introduced by fix for bug 51217

https://gerrit.wikimedia.org/r/78242
Comment 5 ssastry 2013-08-15 01:02:13 UTC
Fix deployed -- cache is not yet purged, but on purge tomorrow, this will be fixed.
Comment 6 ssastry 2013-08-20 20:09:19 UTC
Cache is now purged.  I tested on http://en.wikipedia.org/wiki/Crayon_Shin-chan (another page that had been reported with the bug) and verified fixed.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links