Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX for "mb_detect_encoding() expects parameter 1 to be string, resource given" #13

Open
wants to merge 182 commits into
base: master
Choose a base branch
from

Conversation

spidgorny
Copy link

When one of the debug arrays contains a resource, is_resource() function sometimes does not detect it well.
See http://php.net/manual/en/function.is-resource.php#103942 for details.
This is the fix for the situation.

cadorn and others added 29 commits June 2, 2011 11:39
Add Composer support to FirePHP Core
Modify composer.json in order to load global functions defined at fb.php file
Will now automatically run in tests folder, by just typing ‘phpunit’
PHPUnit: now auto-runs tests without folder argument
Fix is_resource() bug
Conflicts:
	.gitignore
	CHANGELOG.md
	README.md
	lib/FirePHPCore/FirePHP.class.php
	package.json
	workspace/README.md
	workspace/package.json
	workspace/program.json
	workspace/scripts/build.js
	workspace/tpl/readme.tpl.md
@cadorn
Copy link
Member

cadorn commented Mar 17, 2014

You need to send a diff against latest master. This PR has many old changes in it that are already on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants