Commit graph

6 commits

Author SHA1 Message Date
libraryupgrader 613f6f68d4 build: Updating mediawiki/mediawiki-phan-config to 0.5.0
Change-Id: I1d485d6ffd9a8fe7031ee73b634af25628d75c12
2019-03-10 22:21:32 -07:00
jenkins-bot a8932c74bd Merge "No <br /> tag after <hr /> tag to avoid extra space" 2018-06-05 15:10:00 +00:00
Thiemo Mättig 1f6bce0d49 No <br /> tag after <hr /> tag to avoid extra space
if <br /> tag is present after <hr />tag this change replaces both with <hr /> tag. as per Thiemo's patch I8021b2c sugesstions.

Bug: T124144
Change-Id: I9239da4538e189fc825e72668a3cbd8325ac2ca4
2018-05-18 13:19:57 +00:00
Thiemo Mättig 870611c0bb Cleanup and streamline Poem class
* I think the dependency on a more recent PHP version was forgotten.
  Notice how the code already contained short array syntax.
* Make the @license more visible in the class itself.
* I'm intentionally only listing the original @author. Copying this
  author line was forgotten in Ieea24d5.
* Remove not needed "return true" from the hook handler. This is the
  default anyway.
* Use the …::class feature when possible.
* Remove the unwanted &$parser reference. This is only here for legacy
  reasons, but does not make any sense any more (Why would a hook
  handler want to replace the *parser*?), and can be removed from
  callees (not from callers) with no consequence.
* Add strict type hints to function headers for type safety. I checked
  the caller in Parser::extensionSubstitution() that will call
  Poem::renderPoem(), and it is impossible for the Parser to be null,
  and impossible for the frame to be false.

Change-Id: I4b4203bf1cef0c057db5e6a7747d441d0b0e4b54
2018-03-29 19:47:39 +00:00
Kunal Mehta fe52b2318a Add phan configuration
Change-Id: I2479f6ae40cfa410bed3e75a3c3da9bfaf66e41c
2018-02-24 16:53:45 -08:00
Kunal Mehta 1266a9fd0d Move class into includes/
Change-Id: I8e298debf7da3c64769f13aeda91a29646536e3c
2018-02-24 16:51:46 -08:00