-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Laravel 5.6 support #1532
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
Closed
Closed
Laravel 5.6 support #1532
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
setRelationCast add relation that ended with _id into objectId if config allow it
and use HasOneOrManyTrait
Simply override `removeTableFromKey($key)` method and just return original keys in order to support dot-notation when calling `fill` on model.
example: Model::sum('subarray.*.price'); this method is much simpler as to use as complex raw aggregations for this function a $unwind directive will be pushed in pipeline before $group
change test for different scenarios
Fix getOtherKey access level
Replace "otherKey" by "relatedKey"
Solving belongsToMany relationship for Laravel 5.4
This file was missing expire property. I have also changed from DatabaseJob to MongoJob.
Syncing up with jenssegers/laravel-mongodb Queue MongoJob
…in relation will be loaded (See #58).
Many to Many relation maintenance.
Mongo Queue
Fix undefined method getTime
Aggregation shouldn't mutate querybuilder
Add $this->app['hash'] to get token to reset password
Update PasswordBrokerManager.php
Update in function getPayload in created_at Update in function tokenExpired, change the variable $token to $createdAt
Change the the function getPayload and tokenExpired
Update DatabaseTokenRepository.php
Error with new official released Laravel 5.5 Declaration of Moloquent\Eloquent\HybridRelations::belongsToMany($related, $collection = NULL, $foreignKey = NULL, $otherKey = NULL, $relation = NULL) should be compatible with Illuminate\Database\Eloquent\Model::belongsToMany($related, $table = NULL, $foreignPivotKey = NULL, $relatedPivotKey = NULL, $parentKey = NULL, $relatedKey = NULL, $relation = NULL) update to laravel 5.5
Update HybridRelations.php
Gets key name from model rather than assuming _id
Fixes getting relationship by using camelKey instead of key
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Request to update moloquent fork repo failed. Hence accidently requested PR for request to solve #97 of moloquent similar to #1428 pull on parent repo.