Skip to content

fix array to string conversion notices #196

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.

Sign up for GitHub

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

Jump to bottom
Merged
theseer merged 1 commit into from
Jan 25, 2015
Merged

fix array to string conversion notices #196

theseer merged 1 commit into from
Jan 25, 2015

Conversation

@nikolashermann
Copy link
Contributor

@nikolashermann nikolashermann commented Jan 21, 2015

fixes notice:

PHP Version: 5.6.4-1~dotdeb.1 (Linux)
PHPDox Version: c4f9402-dirty
ErrorException: E_NOTICE 
Location: .../vendor/theseer/phpdox/src/collector/project/AbstractUnitObject.php (Line 344)

Array to string conversion

#0 .../vendor/theseer/phpdox/src/collector/project/AbstractUnitObject.php(344): sprintf()
#1 .../vendor/theseer/phpdox/src/collector/InheritanceResolver.php(148): TheSeer\phpDox\Collector\AbstractUnitObject->markDependencyAsUnresolved()
#2 .../vendor/theseer/phpdox/src/collector/InheritanceResolver.php(165): TheSeer\phpDox\Collector\InheritanceResolver->addUnresolved()
#3 .../vendor/theseer/phpdox/src/collector/InheritanceResolver.php(92): TheSeer\phpDox\Collector\InheritanceResolver->processExtends()
#4 .../vendor/theseer/phpdox/src/Application.php(145): TheSeer\phpDox\Collector\InheritanceResolver->resolve()
#5 .../vendor/theseer/phpdox/src/CLI.php(156): TheSeer\phpDox\Application->runCollector()
#6 .../vendor/theseer/phpdox/phpdox(65): TheSeer\phpDox\CLI->run()

theseer added a commit that referenced this pull request Jan 25, 2015
@theseer
Merge pull request #196 from nikolashermann/master
fa8efd8 
fix array to string conversion notices
@theseer theseer merged commit fa8efd8 into theseer:master Jan 25, 2015
@theseer
Copy link
Owner

theseer commented Jan 25, 2015

Nice catch, thank you.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nikolashermann @theseer