Alternative GridFS collection names issue fix. #765

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@3d0c
Contributor

3d0c commented Dec 22, 2012

There is a problem in source/MongoDb.php, it doesn't work with GridFS collections which is named different to defaults.

This call doesn't pass prefix to the getGridFS and defaults "fs.files" will be used, here it is:

$collection = $self->connection->getGridFS();

This PR has a fix for this issue.

But there is one more problem — unable to use different sources for gridfs collection, because of this:

if ($source == "{$_config['gridPrefix']}.files")

I'm afraid It's a wrong way to determine the type of collection — is it gridfs or not.

I'm working now on the solution.

@djordje

This comment has been minimized.

Show comment
Hide comment
@djordje

djordje Dec 22, 2012

Contributor

@3d0c
You should pull request against dev branch.

Contributor

djordje commented Dec 22, 2012

@3d0c
You should pull request against dev branch.

@jails

This comment has been minimized.

Show comment
Hide comment
@jails

jails Dec 23, 2012

Contributor

Close as duplicate #766

Contributor

jails commented Dec 23, 2012

Close as duplicate #766

@jails jails closed this Dec 23, 2012

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