Page MenuHomePhabricator

SkinPerPage should validate the skin name
Open, Needs TriagePublic

Description

If given an invalid skin name, it should return something like

Html::element( 'span', [ 'class' => 'error'], wfMessage( 'skinperpage-noskin' )->text() );

Where skinperpage-noskin is an error message defined in i18n/en.json.

Event Timeline

Bawolff created this task.Aug 30 2018, 7:53 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 30 2018, 7:53 AM
Legoktm updated the task description. (Show Details)

Hello I would like to work on this task. @Bawolff can you please help me to find code base repository and also could you please update the task description.

@Nithinreddy18: Hi and welcome! Please see https://www.mediawiki.org/wiki/Extension:SkinPerPage for information and links to the Gerrit repository.
And if you think that there is a need to update the task description, please explain why you think that there is a need to update the task description. :)

I thought it would be easier if the link for extension and manual adding code in localsettings.php are mentioned in the description.