Page MenuHomePhabricator

Code cleanup
Closed, ResolvedPublic

Description

I did some code cleanup using static code analysis with phpStorm.
Some parts are already reviewd but 3 Changes are still missing.
https://gerrit.wikimedia.org/r/#/q/status:open+project:mediawiki/extensions/Math+branch:master+topic:staticCodeAnalysis,n,z

Event Timeline

Physikerwelt raised the priority of this task from to Needs Triage.
Physikerwelt updated the task description. (Show Details)
Physikerwelt added a project: Math.
Physikerwelt subscribed.
gerritbot subscribed.

Change 178506 had a related patch set uploaded (by Physikerwelt):
Remove unused XML namespace

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

Patch-For-Review

Change 178507 had a related patch set uploaded (by Physikerwelt):
Avoid call to method that could be undefined

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

Patch-For-Review

Change 178508 had a related patch set uploaded (by Physikerwelt):
Add type hinting for newFromMd5 constructor

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

Patch-For-Review

Change 178506 merged by jenkins-bot:
Remove unused XML namespace

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

Change 178508 merged by jenkins-bot:
Add type hinting for newFromMd5 constructor

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

Change 178507 abandoned by Krinkle:
Avoid call to method that could be undefined

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

Krinkle claimed this task.
Krinkle subscribed.

Done.