Page MenuHomePhabricator

lang="php-brief" broken
Closed, ResolvedPublic

Description

API:Calling_internally had bad-looking PHP examples. They were using

<source lang="php-brief">

$params = new DerivativeRequest( 
        $this->getRequest(),
        array(
       ...
</source>

The generated HTML has CSS spans in it like <div class="php-brief source-php-brief"> ... <span class="kw3">array</span>... </div>, but there are no styles for php-brief so the code isn't colored. And there's some weird framing.

The workaround is easy, just switch to lang=php.

To fix maybe lang=php-brief should request its own ext.geshi.language.php-brief ResourceLoader styles module or reuse the php CSS, but it's probably easier to just deprecate it and/or treat php-brief as a synonym for lang=php.

Meanwhile I've deprecated php-brief in https://www.mediawiki.org/wiki/Extension:SyntaxHighlight_GeSHi

Event Timeline

Spage raised the priority of this task from to Low.
Spage updated the task description. (Show Details)
Spage added a project: SyntaxHighlight.
Spage subscribed.

Change 200302 had a related patch set uploaded (by TheDJ):
Add missing and new languages

https://gerrit.wikimedia.org/r/200302

Although I support deprecating this language, I was sort of surprised by this.
All style modules for geshi are auto generated so I suspected something more was going on. And indeed something was going on, there was a small problem in the list generator (which also had not been run to update the supported language list after the last update.

Changed the script, regenerated the list of supported languages.

If we want to remove languages, we should probably add a 'black list' and diff the arrays to exclude whatever we don't want to use.

Change 200302 merged by jenkins-bot:
Add missing and new languages

https://gerrit.wikimedia.org/r/200302

matmarex renamed this task from lang="php-brief" broken, remove it? to lang="php-brief" broken.Apr 10 2015, 12:17 PM
matmarex closed this task as Resolved.
matmarex assigned this task to TheDJ.
matmarex removed a project: Patch-For-Review.
matmarex set Security to None.