"Nikerabbit" changed the status of MediaWiki.r107625 to "fixme" and commented 
it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/107625#c28676

Old Status: new
> New Status: fixme

Commit summary for MediaWiki.r107625:

now beta - cleaned up code - added to credits - time comparison code still 
needs revamp - ready for feedback and tweaks welcome

Nikerabbit's comment:

<pre>                   foreach ( $res as $row ) {
                                $edittime[] = $row->rev_timestamp;
                        }
+                       $edittime = array();
+                       foreach ( $res as $row ) {
+                               $edittime[] = $row->rev_timestamp;
+                       }
</pre>

Looks like broken merge.

There are some very long lines in this commit.

 +                      $response = file_get_contents( $bitly );
We have Http::get/post for doing that. And the function name and comment seem 
to be outdated.


 +                      $tweet_text = substr( $tweet_text, 0, $tweet_text_count 
);
Use $language::truncate() to do it in multibyte safe way.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to