Page MenuHomePhabricator

Fix typo in GeSHi HTML5 definition file
Closed, ResolvedPublic

Description

[mediawiki/extensions/SyntaxHighlight_GeSHi.git] / geshi / geshi / html5.php

End of line 61 is "." and should be "," instead.


Version: unspecified
Severity: trivial
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=37585

Details

Reference
bz37044

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:28 AM
bzimport added a project: SyntaxHighlight.
bzimport set Reference to bz37044.

These should be reported to upstream http://qbnz.com/highlighter/ and ask here to update the version of bundled geshi once fixed.

Although it's upstream, we can simply fix it ourselves.

Besides the demo on the site hilites correctly, so it obviously has already been fixed in some newer version we don't use yet.

There is no html5.php file in GeSHi standard distribution.

Furthermore, their bugtracker isn't available for the moment: http://geshi.org/developers/bugs/

Thank you for your report.

Code review: https://gerrit.wikimedia.org/r/#/c/10286/


Danny B., please read http://www.mediawiki.org/wiki/Developer_access if you wish to join the MediaWiki development community and be able to add yourself patches in our code review queue.

You can get an overview of the process on http://www.mediawiki.org/wiki/Git/Workflow

I used the following commands to put your patch in our community code review site:

git clone ssh://review/mediawiki/extensions/SyntaxHighlight_GeSHi.git
cd SyntaxHighlight_GeSHi
git checkout -b geshi-37044
(editing the relevant file)
git add geshi/geshi/html5.php
git commit -a -m "(bug 37044) Fixed HTML5 syntax highlight typo"
git review

Even though geshi library is now physically bundled in the repo, we shouldn't be making changes to it.

Okay,

I've made further inquiries.

There is two versions of GeSHI:

  • 1.0, with the HTML5.php file
  • 1.1, currently in development

The correct one is the 1.0.X, as indicated in extension README file.
http://geshi.svn.sourceforge.net/viewvc/geshi/trunk/geshi-1.0.X/

The typo reported by Danny B. have been fixed 8 months ago:
http://geshi.svn.sourceforge.net/viewvc/geshi/trunk/geshi-1.0.X/src/geshi/html5.php?r1=2430&r2=2479

The GeSHI project doesn't seem very active currently.

I opened the bug #37585 to track the upcoming release and fix it.