Last modified: 2014-08-05 20:25:28 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68448, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66448 - Warning: trim() expects parameter 1 to be string, array given in OggHandler.php on line 116
Warning: trim() expects parameter 1 to be string, array given in OggHandler.p...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
TimedMediaHandler (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-10 19:55 UTC by Sam Reed (reedy)
Modified: 2014-08-05 20:25 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2014-06-10 19:55:32 UTC
Warning:  trim() expects parameter 1 to be string, array given in /usr/local/apache/common-local/php-1.24wmf8/extensions/TimedMediaHandler/handlers/OggHandler/OggHandler.php on line 116
Comment 1 Sam Reed (reedy) 2014-06-18 17:40:33 UTC
			foreach( $stream['comments'] as $name => $value ) {
				$trimmedValue = trim( $value );
				if ( $trimmedValue === '' ) {
					continue;
				}


Did we start allowing/passing/parsing in arrays at some point?

I guess it needs special handling
Comment 2 Gerrit Notification Bot 2014-06-19 04:03:18 UTC
Change 140641 had a related patch set uploaded by Brian Wolff:
Handle double tags properly (fix php warning about trim())

https://gerrit.wikimedia.org/r/140641
Comment 3 Gerrit Notification Bot 2014-06-19 15:16:44 UTC
Change 140641 merged by jenkins-bot:
Handle double tags properly (fix php warning about trim())

https://gerrit.wikimedia.org/r/140641
Comment 4 Tomasz W. Kozlowski 2014-08-05 20:25:28 UTC
Patch was merged, so assuming bug is fixed.

Please re-open if that's not the case.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links