Don't load extension or skin when marked false
ClosedPublic
Actions

Authored by dereckson on Mar 22 2018, 23:41.

Tags

None

Referenced Files

	F51754745: D1475.diff
	Sun, Sep 27, 22:06

	Unknown Object (File)
	Sat, Sep 26, 19:11

	Unknown Object (File)
	Sat, Sep 26, 08:53

	Unknown Object (File)
	Wed, Sep 23, 05:24

	Unknown Object (File)
	Wed, Sep 16, 22:23

	Unknown Object (File)
	Tue, Sep 15, 22:45

	Unknown Object (File)
	Fri, Sep 11, 08:18

	Unknown Object (File)
	Fri, Sep 11, 06:32

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rSAASMW17d806c8e303: Don't load extension or skin when marked false

Summary

The saasUse{Type}{Name} setting is a boolean requiring to load
an extension or a skin.

The extension or skin was currently unconditionally loaded,
as the loader only checked the presence of the key, not its
value.

Test Plan

Tested with D1474

Diff Detail

Repository

rSAASMW SaaS MediaWiki

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Mar 22 2018, 23:41

dereckson created this revision.

Harbormaster completed remote builds in B2330: Diff 3780.Mar 22 2018, 23:41

dereckson accepted this revision.Mar 22 2018, 23:41

dereckson added a child revision: D1474: Enable WolfplexMessages on www.wolfplex.org.

This revision is now accepted and ready to land.Mar 22 2018, 23:42

Closed by commit rSAASMW17d806c8e303: Don't load extension or skin when marked false (authored by dereckson). · Explain WhyMar 22 2018, 23:44

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

Configuration.php

2 lines

Diff 3781

View Options

Don't load extension or skin when marked falseClosedPublicActions