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

I haven't tested this in enough varying circumstances, but this fixed a number of failure-to-execute problems for me... #2

Closed
wants to merge 1 commit into from

Conversation

jonphipps
Copy link
Contributor

This now checks for the existence of the autoload files either in the current
path or the include_path before loading.

In addition, mink/autoload.php loads 'vendor/.composer/autoload.php'
and if mink exists then composer/autoload.php tries to redeclare the class. This catches that.

Not tested beyond 'Works for me!'

This now checks for the existence of the files either in the current
path or the include_path before loading.

In addition, mink/autoload.php loads 'vendor/.composer/autoload.php'
and if mink and if it exits then it doesn't need to load again.

Not tested beyond 'Works for me!'
@DavertMik
Copy link
Member

It's in new release 1.0.3. Thanks for help.

@DavertMik DavertMik closed this Feb 14, 2012
DavertMik pushed a commit that referenced this pull request Feb 8, 2013
* Proper codeformatting
* Using collection::drop() instead of deprecated Mongo::dropCollection()
* checking collection status before droping to fix test
DavertMik added a commit that referenced this pull request May 24, 2014
Naktibalda pushed a commit that referenced this pull request Nov 22, 2015
Extend unit tests in testRegexFilter method
@tasselchof tasselchof mentioned this pull request May 3, 2017
DavertMik pushed a commit that referenced this pull request Oct 4, 2018
* Fixes PHPStan issue

Uses full namespace for docblock, allowing PHPStan to find the class when using namespaces

* Fixes docblock (#2)

* Alphabetize types
Naktibalda pushed a commit that referenced this pull request Jan 2, 2021
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.

None yet

2 participants