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 fatal syntax error "unexpected '-', expecting '{'" #3219

Closed
wants to merge 1 commit into from
Closed

Fix fatal syntax error "unexpected '-', expecting '{'" #3219

wants to merge 1 commit into from

Conversation

Hecke29
Copy link
Contributor

@Hecke29 Hecke29 commented Jul 19, 2018

caused by special chars in wsdl filename

This might also fix sebastianbergmann/phpunit-mock-objects#357 and fix #2573

Stumbled over that bug yesterday. Remote file could be load via http (http://example.org/soap.wsdl) but saved as local file with name (/home/user/my-dir/acme-soap.wsdl) lead to that error.

@codecov-io
Copy link

codecov-io commented Jul 19, 2018

Codecov Report

Merging #3219 into 7.2 will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##                7.2    #3219      +/-   ##
============================================
+ Coverage     81.43%   81.43%   +<.01%     
  Complexity     3393     3393              
============================================
  Files           137      137              
  Lines          8978     8979       +1     
============================================
+ Hits           7311     7312       +1     
  Misses         1667     1667
Impacted Files Coverage Δ Complexity Δ
src/Framework/TestCase.php 75.89% <100%> (+0.02%) 307 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0102d46...1009f7b. Read the comment docs.

@sebastianbergmann
Copy link
Owner

Cherry-picked into 7.3 and merged to master from there.

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

Successfully merging this pull request may close these issues.

3 participants