-
Notifications
You must be signed in to change notification settings - Fork 356
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
Convert from Laminas\Db to Doctrine #2233
Draft
demiankatz
wants to merge
371
commits into
vufind-org:dev
Choose a base branch
from
demiankatz:doctrine
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 34 commits
Commits
Show all changes
371 commits
Select commit
Hold shift + click to select a range
674df7f
Merge branch 'dev' into doctrine
demiankatz 595a747
Fix boolean compatibility for PostgreSQL.
demiankatz f74f2af
Merge branch 'dev' into doctrine
demiankatz df403da
Merge branch 'dev' into doctrine
demiankatz 6b46209
Fix PHP version in comments.
demiankatz 75864ea
Merge branch 'dev' into doctrine
demiankatz c604833
Fix cache initialization.
demiankatz 1417509
Fix PHPUnit 10 compatibility.
demiankatz bbc8a37
Fix favorites logic.
demiankatz 3829157
Do not access property on boolean.
demiankatz ac4e4dd
Fix problems with secure delegator.
demiankatz ed8b649
Fix PHPUnit 10 compatibility.
demiankatz c85e27b
Merge branch 'dev' into doctrine
demiankatz 97b26b9
Improve catalog credential / library card code.
demiankatz 318a75d
Merge branch 'dev' into doctrine
demiankatz 1598e11
Fix InstallController.
demiankatz 6c6f892
Port LibraryCard logic in SwitchDbHashCommand.
demiankatz 2fcf884
Merge branch 'dev' into doctrine
demiankatz 171b628
More specific return types.
demiankatz 978fdc2
Simplify use statement.
demiankatz 18c51b3
Merge branch 'dev' into doctrine
demiankatz 5e08683
Add encrypt/decrypt wrapper methods for clarity.
demiankatz 9e5b472
Rename database service base class/factory.
demiankatz 184f528
More accurate typehint.
demiankatz 81fd55c
Rename service aware trait/interface; clean up namespacing.
demiankatz 16b2bd7
Merge branch 'dev' into doctrine
demiankatz e0e3011
Integrate EntityInterface with UserEntityInterface.
demiankatz 5f94d99
Merge branch 'dev' into doctrine
demiankatz e8a3ba1
Merge branch 'dev' into doctrine
demiankatz 5bcdd72
Improve comment.
demiankatz c795a90
Change return types to use interface.
demiankatz 69be0c0
Merge branch 'dev' into doctrine
demiankatz d46dc14
Add missing methods.
demiankatz bcf9747
Merge commit 'cf7f140cd99a9ec1e46e9667e078be14ef45addf' into doctrine
demiankatz ce896ed
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz b1379f4
Merge branch 'dev' into doctrine
demiankatz ba4d973
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 3b7d5af
Merge branch 'dev' into doctrine
demiankatz b7e01f9
Merge branch 'dev' into doctrine
demiankatz d78d6cb
Merge branch 'dev' into doctrine
demiankatz 9fe7c57
Merge branch 'dev' into doctrine
demiankatz eb04df7
Begin filling in AccessToken methods.
demiankatz cddaf67
Entity cleanup.
demiankatz e4ba78f
Merge branch 'dev' into doctrine
demiankatz b5caf38
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 565a5f1
Merge branch 'dev' into doctrine
demiankatz 48a8803
Merge branch 'dev' into doctrine
demiankatz 3af2d9c
Move, simplify and type addComment method.
demiankatz 2b72898
Fix case typo.
demiankatz 28aced6
Merge branch 'dev' into doctrine
demiankatz 6b1a923
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 44279fb
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 00d7249
Merge branch 'dev' into doctrine
demiankatz 7587aa7
php-cs-fixer
demiankatz c23c16a
Adjust parameter order; simplify code.
demiankatz f5ae758
Merge branch 'dev' into doctrine
demiankatz 824cb31
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 05aac84
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 6ab866b
Merge branch 'dev' into doctrine
demiankatz 0162a5d
Minor CommentsService improvements.
demiankatz fba1b81
More simplification.
demiankatz d5ca090
Merge branch 'dev' into doctrine
demiankatz 52ebbea
Merge branch 'dev' into doctrine
demiankatz b171451
Make OaiResumptionService::encodeParams() protected.
demiankatz d838806
Simplify and modernize test.
demiankatz da4f8e2
Merge branch 'dev' into doctrine
demiankatz ce45f61
Merge branch 'dev' into doctrine
demiankatz 45012d6
Merge branch 'dev' into doctrine
demiankatz 33f68ea
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 8cc38d2
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz f30961a
Expand use of getDoctrineReference; improve types.
demiankatz b758443
Merge branch 'dev' into doctrine
demiankatz 1a177a9
Merge branch 'dev' into doctrine
demiankatz 57e4173
Merge branch 'dev' into doctrine
demiankatz ba68950
Add new methods to entity.
demiankatz 8090b5a
Merge branch 'dev' into doctrine
demiankatz baf46ea
Merge branch 'dev' into doctrine
demiankatz 7cc3e87
Merge branch 'dev' into doctrine
demiankatz 5e89774
Fix typo.
demiankatz e5b3136
Merge branch 'dev' into doctrine
demiankatz 7b28efc
Merge branch 'dev' into doctrine
demiankatz 8c3f509
Add missing method.
demiankatz 628f095
Merge branch 'dev' into doctrine
demiankatz 7cfac38
Roll back unnecessary change.
demiankatz c4dd770
Merge branch 'dev' into doctrine
demiankatz ca475d8
Clean up namespace usage.
demiankatz 84e72ad
Improve editAllowed().
demiankatz 2ab0644
Merge branch 'dev' into doctrine
demiankatz 5234e6c
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz da00f37
Fix case of DateTime class.
demiankatz c1ca391
Merge branch 'dev' into doctrine
demiankatz 70ac464
Add missing interface.
demiankatz babb53f
Remove unwanted throw.
demiankatz 19be38f
Remove obsolete reference.
demiankatz 3c5d0cf
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz eade5c4
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 861290e
Merge branch 'dev' into doctrine
demiankatz 4043c57
Code simplifications.
demiankatz 2d2ca1a
Fix broken test.
demiankatz 44042cf
Merge branch 'dev' into doctrine
demiankatz 14e57fa
Merge branch 'dev' into doctrine
demiankatz 5cbe584
Merge branch 'dev' into doctrine
demiankatz 453a289
Update bootstrap5 theme.
demiankatz 0ff0d0a
Merge branch 'dev' into doctrine
demiankatz be1b9ce
Merge branch 'dev' into doctrine
demiankatz 2ce9f85
Fix merge error.
demiankatz 98ceede
Merge branch 'dev' into doctrine
demiankatz a1a9848
Merge branch 'dev' into doctrine
demiankatz 1261085
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 7274bf2
Merge branch 'dev' into doctrine
demiankatz 691838e
Add new interface methods to AuthHash entity.
demiankatz e0f9837
Merge branch 'dev' into doctrine
demiankatz f3e14e3
Implement ExternalSessionEntityInterface in entity class.
demiankatz 0823739
Merge branch 'dev' into doctrine
demiankatz c9b3594
Merge branch 'dev' into doctrine
demiankatz 39592d7
Merge branch 'dev' into doctrine
demiankatz 25379a8
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 5c989be
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz d04d2e8
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz dde6b4e
Clean up comments.
demiankatz 72646d5
More cleanup.
demiankatz f922264
Merge branch 'dev' into doctrine
demiankatz ee4bd71
Add testing of deleted records to OaiTest.
demiankatz 3e2a614
Remove debug line.
demiankatz 7a18a96
Merge branch 'dev' into doctrine
demiankatz 0259d61
Merge branch 'oai-deleted-test' into doctrine
demiankatz de69786
Fix broken test.
demiankatz 61e37dc
Merge branch 'dev' into doctrine
demiankatz 5f068d9
Merge branch 'dev' into doctrine
demiankatz 931156d
Remove unused method and associated test.
demiankatz bfd275c
php-cs-fixer
demiankatz 15aa44e
Merge branch 'dev' into doctrine
demiankatz 03bf4b9
Minor fix.
demiankatz e1c1c49
Merge branch 'dev' into doctrine
demiankatz 56c60df
Reduce diffs.
demiankatz a5c6571
Clarify feedback templates.
demiankatz 44c7f60
Fix test.
demiankatz 83ad677
Merge branch 'dev' into doctrine
demiankatz b9115c2
Merge branch 'dev' into doctrine
demiankatz 37ffd6f
Merge branch 'dev' into doctrine
demiankatz 129214a
phpcbf
demiankatz 9be382c
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 30ae6d0
Merge branch 'dev' into doctrine
demiankatz 4654027
Merge branch 'dev' into doctrine
demiankatz 2749e06
Merge branch 'dev' into doctrine
demiankatz de77327
Merge branch 'dev' into doctrine
demiankatz 372b954
Merge branch 'dev' into doctrine
demiankatz e4c8704
Merge branch 'dev' into doctrine
demiankatz f7601cc
Merge branch 'dev' into doctrine
demiankatz 31f40ae
Add missing return.
demiankatz 1828a00
Merge branch 'dev' into doctrine
demiankatz 31db1e1
Merge branch 'dev' into doctrine
demiankatz ac1d867
Merge branch 'dev' into doctrine
demiankatz 9ac7c41
php-cs-fixer
demiankatz 6a5c793
Merge branch 'dev' into doctrine
demiankatz 5496dc7
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz acbeed1
Merge branch 'dev' into doctrine
demiankatz c6e2ad9
Merge branch 'dev' into doctrine
demiankatz 165dd62
Make \VuFind\Db\ConnectionFactory easier to extend (#30)
aleksip 3e0a4bf
Merge branch 'dev' into doctrine
demiankatz e409022
Merge branch 'dev' into doctrine
demiankatz 8006615
Merge branch 'dev' into doctrine
demiankatz 48bac86
Merge branch 'dev' into doctrine
demiankatz 2c163d2
Merge branch 'dev' into doctrine
demiankatz b5038b4
Remove unused code.
demiankatz b560761
Fix alphabetical order.
demiankatz c54034d
Revert unnecessary change.
demiankatz 1d5c483
Merge branch 'dev' into doctrine
demiankatz 3a2ee02
Merge branch 'dev' into doctrine
demiankatz c2206cc
Merge branch 'dev' into doctrine
demiankatz 4fac62d
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 3ccfa60
Fix merge glitch.
demiankatz 17f453d
php-cs-fixer.
demiankatz d0dec6a
Reduce unnecessary diffs.
demiankatz 99b25f8
Make method protected since it's not used externally.
demiankatz c9e0592
Merge branch 'dev' into doctrine
demiankatz 761c935
Merge branch 'dev' into doctrine
demiankatz 4fdeff3
Merge branch 'dev' into doctrine
demiankatz 6dabb82
Merge branch 'dev' into doctrine
demiankatz a892f12
Merge branch 'dev' into doctrine
demiankatz 68289c8
Merge branch 'dev' into doctrine
demiankatz 7a1012a
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 94e6c27
Minor user authentication improvements (follows #3625).
demiankatz 3ca1f52
Merge branch 'user-auth-fixes' into doctrine
demiankatz f0f0087
php-cs-fixer.
demiankatz ed30fe4
Minor fix.
demiankatz 0be4ee9
php-cs-fixer
demiankatz e370658
Reduce diffs.
demiankatz 5f22be8
More diff reduction/simplification.
demiankatz 62d0024
Merge branch 'dev' into doctrine
demiankatz da3f5fb
Merge branch 'dev' into doctrine
demiankatz 59d2afa
Merge branch 'dev' into doctrine
demiankatz 15ea6e1
Merge branch 'dev' into doctrine
demiankatz e4d23b7
Merge branch 'dev' into doctrine
demiankatz 29a65c8
Merge branch 'dev' into doctrine
demiankatz 74c1aaa
Merge branch 'dev' into doctrine
demiankatz eb53990
Fix tag sorting.
demiankatz 9d384c4
Fix phpstan issue.
demiankatz 5bb8928
Merge branch 'dev' into doctrine
demiankatz a8af487
Merge branch 'dev' into doctrine
demiankatz e2dd538
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz 330d8b0
Fix bug.
demiankatz e9cd529
Merge branch 'dev' into doctrine
demiankatz a710e35
Merge branch 'dev' into doctrine
demiankatz 3026b96
Merge branch 'dev' into doctrine
demiankatz 09a423d
Style fixes.
demiankatz 0a36939
Fix test.
demiankatz 598d46d
More fixes.
demiankatz 52d419e
One last fix.
demiankatz b2406c4
Remove deprecated method.
demiankatz eeffc8b
Merge branch 'dev' into doctrine
demiankatz b61ce9b
Merge branch 'dev' into doctrine
demiankatz 5eb2d60
Merge branch 'dev' into doctrine
demiankatz 4d64f77
Reduce redundancy.
demiankatz 16c7476
Merge branch 'dev' into doctrine
demiankatz 3a6f234
Merge branch 'dev' into doctrine
demiankatz f14fe41
Merge branch 'dev' into doctrine
demiankatz a16a9dc
Merge remote-tracking branch 'origin/dev' into doctrine
demiankatz e7dce70
Merge branch 'dev' into doctrine
demiankatz d6cc727
Merge branch 'dev' into doctrine
demiankatz 80c36dc
Merge branch 'dev' into doctrine
demiankatz 02e7666
Bug fix.
demiankatz 2ed20f0
Merge branch 'dev' into doctrine
demiankatz ba5fe82
Merge branch 'dev' into doctrine
demiankatz f4d4f23
Merge branch 'dev' into doctrine
demiankatz 3ded82b
Merge branch 'dev' into doctrine
demiankatz ea58ce1
Fix issues.
demiankatz 9fb5494
LoginToken Doctrine entity class (#31)
padmasreegade ae159c1
Merge branch 'dev' into doctrine
demiankatz 17bae4d
Merge branch 'dev' into doctrine
demiankatz 147a6cd
Merge branch 'dev' into doctrine
demiankatz 19ece32
Merge branch 'dev' into doctrine
demiankatz a6796f3
Merge branch 'dev' into doctrine
demiankatz 99df79f
Fix factory to use current interfaces.
demiankatz 845df3d
Merge branch 'dev' into doctrine
demiankatz 8916731
Merge branch 'dev' into doctrine
demiankatz 4951130
Merge branch 'dev' into doctrine
demiankatz 9e0c5c2
Access token service doctrine (#32)
padmasreegade 60966f8
Merge branch 'dev-11.0' into doctrine
demiankatz 26e19e6
Merge branch 'dev-11.0' into doctrine
demiankatz 3ef5c51
Merge branch 'dev-11.0' into doctrine
demiankatz c21d9a6
Add SecretTrait support to database ConnectionFactory.
demiankatz 0b04e24
Auth hash service doctrine (#33)
padmasreegade 761a5f5
Merge branch 'dev-11.0' into doctrine
demiankatz 6fb9c88
Merge branch 'dev-11.0' into doctrine
demiankatz 8ad9a03
Merge branch 'dev' into doctrine
demiankatz e66a394
Merge branch 'dev' into doctrine
demiankatz 5bc9ede
Merge branch 'dev' into doctrine
demiankatz 40c8613
Merge branch 'dev' into doctrine
demiankatz 9b86ddc
Migration of Login Token Service from Laminas to Doctrine. (#35)
padmasreegade File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
<?php | ||
|
||
$app = include __DIR__ . '/application.php'; | ||
return \Doctrine\ORM\Tools\Console\ConsoleRunner::createHelperSet( | ||
$app->getServiceManager()->get('doctrine.entity_manager.orm_default') | ||
); |
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aleksip, I see that you deleted this factory, but the configuration is still here. We should clean something up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, forgot to remove that line. Should I push the change to your branch or what would be the best way to work on this together?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please feel free to push directly here, as long as the code still works. I'm not going to be working on this much until next year, so I welcome any progress you can make in the meantime, and I don't anticipate that we'll "step on each other," so to speak.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, if at any point it would be helpful to get a second opinion or anything, or if you would like me to run any tests, just let me know and I'll be happy to help!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@demiankatz I meant to push the changes to my repository and then point you there. Not sure how multi-user PRs should work... Should I push future suggested changes to my clone so you can then decide whether to merge to your branch from there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aleksip, it is possible to make a pull request against a pull request -- you can just target a PR against this branch. That might be the easiest workflow going forward, especially when multi-user collaboration picks up... but since things are quiet right now, feel free to work directly in this branch, at least until you get things into a correct/stable state again. (I think it's easier to just move forward and fix things than to roll back changes and make a separate PR, at least under the present circumstances).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@demiankatz Seems like I can create a PR for the branch in your repository! Should I do that? Edit: missed your above reply.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you want to try it as a proof-of-concept, feel free... but otherwise, as I said, you can just go ahead and push. :-)