wikimedia/mediawiki-extensions-DiscussionTools
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools synced 2024-12-18 02:51:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "CommentItem/HeadingItem: Make more constructor args required"

Browse source
This commit is contained in:
jenkins-bot 2021-01-08 20:59:24 +00:00 committed by Gerrit Code Review
parent 89f6756e16 9ba6c3d159
commit 452c6401d7
2 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
includes/CommentItem.php
View file

@ -20,12 +20,12 @@ class CommentItem extends ThreadItem {
* timestamps) for this comment. There is always at least one signature, but there may be
* multiple. The author and timestamp of the comment is determined from the first signature.
* The last node in every signature range is a node containing the timestamp.
* @param string|null $timestamp
* @param string|null $author Comment author's username
* @param string $timestamp
* @param string $author Comment author's username
*/
public function __construct(
int $level, ImmutableRange $range,
array $signatureRanges = [], ?string $timestamp = null, ?string $author = null
array $signatureRanges, string $timestamp, string $author
) {
parent::__construct( 'comment', $level, $range );
$this->signatureRanges = $signatureRanges;

3
tests/phpunit/ThreadItemTest.php
View file

@ -25,8 +25,7 @@ class ThreadItemTest extends CommentTestCase {
$makeThreadItem = function ( array $arr ) use ( &$makeThreadItem, $range ) : ThreadItem {
if ( $arr['type'] === 'comment' ) {
$item = new CommentItem( 1, $range );
$item->setAuthor( $arr['author'] );
$item = new CommentItem( 1, $range, [], 'TIMESTAMP', $arr['author'] );
} else {
$item = new HeadingItem( $range, 2 );
}