Skip to content

Conversation

m7moud
Copy link
Contributor

@m7moud m7moud commented Jan 13, 2015

No description provided.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.19%) when pulling 765898c on m7moud:master into 007aac0 on sokil:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.19%) when pulling 765898c on m7moud:master into 007aac0 on sokil:master.

@m7moud m7moud changed the title updateAll and updateMultiple fix #71 updateAll and updateMultiple fix Jan 13, 2015
@m7moud
Copy link
Contributor Author

m7moud commented Jan 13, 2015

Or you can Update the README, I just noticed that php-mongo-migrator README is different.

@sokil
Copy link
Owner

sokil commented Jan 13, 2015

Operator class has more than one method, and you may set it through array too:

<?php

$collection->update([], ['k' => 'v']);
$collection->update([], ['$set' => [['k' => 'v']]);
$collection->update([], ['$push' => [['k' => 'v']]);

This bug is fixed in v.1.12.2, can you test it?

And thank you for youtr help and contributions

@sokil
Copy link
Owner

sokil commented Jan 13, 2015

php-mongo-migrator - this tool is for migrations only. it based on this library but used to migrate schamas of documents

@sokil sokil closed this Jan 13, 2015
@m7moud
Copy link
Contributor Author

m7moud commented Jan 13, 2015

I know that php-mongo-migrator is for migrations, but it's README uses the update by passing an operator and not an array, that's what I meant.

@sokil
Copy link
Owner

sokil commented Jan 13, 2015

Now i understand. Second operator accept array (as mongo driver), Operator object as in sample in php-mongo-migrator and callable that conficure operator. For be callable is the bast case.

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