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

Pull request to test value before using function ctype #208

Closed
wants to merge 45 commits into from

Conversation

Alexandre-T
Copy link

@Alexandre-T Alexandre-T commented Feb 21, 2020

This pull request ws created to Fix #207. But when I saw the reports of codeclimat, the problem with travis, I decided to fork the project to purpose a new release of doctrine2-spatial which can works with the current PHP versions and doctrine versions.

@coveralls
Copy link

Coverage Status

Coverage decreased (-43.0%) to 55.041% when pulling c6e9471 on Alexandre-T:master into 58ea5fa on creof:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-43.0%) to 55.041% when pulling c6e9471 on Alexandre-T:master into 58ea5fa on creof:master.

@coveralls
Copy link

coveralls commented Feb 21, 2020

Coverage Status

Coverage decreased (-0.4%) to 97.605% when pulling c35c6da on Alexandre-T:master into 58ea5fa on creof:master.

@holtkamp
Copy link

Great job working on a new fork, also got some change in my fork which might be useful, will have a look after my vacation ;)

@Alexandre-T
Copy link
Author

I close this pull request, because my fork will create some BC incompatibility, but i will submit a pull request for a next major version when upgrade will be done.

@Alexandre-T
Copy link
Author

Hello @holtkamp , I have published a 2.0.0-rc.0 version. If you want, you can test it and install it via composer (Be careful to specify the 2.0.0-rc.0 version)

composer require "alexandret/doctrine2-spatial:2.0.0-rc.0"

A full documentation can be found here: https://doctrine2-spatial.readthedocs.io/en/latest/

Please forgive me, I'm not fluent in english.

@holtkamp
Copy link

Great work @Alexandre-T, will try to have a look coming weeks and report my findings.

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.

Trying to access array offset on value of type resource
3 participants