From: Matěj Suchánek <matejsuchanek97@gmail.com>
Date: Sun, 29 Jul 2018 09:54:45 +0000 (+0200)
Subject: Treat all time values on Special:Watchlist as floats
X-Git-Tag: 1.34.0-rc.0~4628^2
X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/banques/ajouter.php?a=commitdiff_plain;h=4b75063d0eafeb9260f177cde782156a81b56f6b;p=lhc%2Fweb%2Fwiklou.git

Treat all time values on Special:Watchlist as floats

PHP division may return float or int, depending on the operands. [1]
Make sure all numbers are of the same type (floats), because
XmlSelect compares values with ===.

[1] http://php.net/manual/en/language.operators.arithmetic.php

Bug: T199566
Change-Id: I37df6fd425f47d9a4562d83e04fcb50c3b97e0da
---

diff --git a/includes/specials/SpecialWatchlist.php b/includes/specials/SpecialWatchlist.php
index 41a059fba4..5b48f4ef46 100644
--- a/includes/specials/SpecialWatchlist.php
+++ b/includes/specials/SpecialWatchlist.php
@@ -777,7 +777,7 @@ class SpecialWatchlist extends ChangesListSpecialPage {
 		] ) );
 		asort( $hours );
 
-		$select = new XmlSelect( 'days', 'days', $selectedHours / 24 );
+		$select = new XmlSelect( 'days', 'days', (float)( $selectedHours / 24 ) );
 
 		foreach ( $hours as $value ) {
 			if ( $value < 24 ) {
@@ -785,7 +785,7 @@ class SpecialWatchlist extends ChangesListSpecialPage {
 			} else {
 				$name = $this->msg( 'days' )->numParams( $value / 24 )->text();
 			}
-			$select->addOption( $name, $value / 24 );
+			$select->addOption( $name, (float)( $value / 24 ) );
 		}
 
 		return $select->getHTML() . "\n<br />\n";