Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Roadmap
- Improve unit test coverage.
Add migration templates for Drupal 6 and Drupal 7.Support multiple roles- Doneviews integration- Done
Comment | File | Size | Author |
---|---|---|---|
#27 | interdiff-2883617-26-27.txt | 1.5 KB | mradcliffe |
#27 | agreement-2883617-test-agreement-handler-27.patch | 9.53 KB | mradcliffe |
| |||
#26 | agreement-2883617-test-agreement-handler-26.patch | 9.31 KB | mradcliffe |
| |||
#23 | interdiff-2883617-21-22.txt | 7.44 KB | mradcliffe |
#22 | agreement-2883617-migrate-22.patch | 28.41 KB | mradcliffe |
|
Comments
Comment #2
mradcliffeThank you for your create the issue and your efforts @akashjain132. I appreciate it. I actually have a work-in-progress 8.x-2.x branch in my private repository that incorporates some of my planned architectural changes in #1852834: Possible to have multiple agreements - role based. I had not gotten around to pushing it as I had started working on it before I became a maintainer.
It will probably take me some time to review your work on github to compare it to what I have already, but I would like to try to incorporate any improvements you have there into my own code (with attribution).
At the moment my priority in my free time is to try to incorporate and deal with the merge conflicts around the following feature and bug requests:
- #1852834: Possible to have multiple agreements - role based
- #2873904: Allow more agreement frequency options
- #2374539: Multiple Roles
- #2879720: avoid Drupal API function calls in global context
I don't want to prematurely push the architecture into an official 8.x-2.x branch yet, but I should probably push my 8.x-2.x-WIP branch upstream here at least.
Comment #3
mradcliffeBecause I don't have version control access on the agreement project I've pushed my 8.x-2.x work into a public github repository, https://github.com/mradcliffe/agreement.
I would probably re-fork from my repository, and then add some pull requests until we can create a functional patch to squash down here.
Roadmap:
- Re-implement functional tests as FunctionalJavascript tests.
- Add migration templates for Drupal 6 and Drupal 7.
Comment #4
mradcliffeComment #5
mradcliffeThis is functional with tests now on github.
Next step is to get access to create branches on drupal.org so that I can move things back here.
Comment #6
mradcliffeOkay, 8.x-2.x branch created and here's a patch from GitHub.
- Also implements #2374539: Multiple Roles with new functional tests.
- Functional tests ported to PHPUnit.
- Unit test added.
Still needs views integration.
Comment #8
mradcliffeAdds views integration back.
Comment #10
mradcliffeFixes views schema so that options are set as a sequence instead of a mapping.
Comment #11
mradcliffeLet's get those code standards fixed while I'm at it too.
Comment #13
mradcliffeUpdating issue summary to include recently done work.
Comment #14
mradcliffeHere's the start of some migration templates and plugins with a test and fixture that will probably fail.
Comment #15
mradcliffeStill not working, but I forgot the destination plugin for agreement table, which I think I'm doing what I'm supposed to be doing but the API documentation is not really clear.
Comment #18
mradcliffeFixes the fixture issue and gets it almost there.
- Agreement type migration is working.
- Agreement migration is not working. For some reason even though the agreement destination plugin is firing the import method, nothing is getting inserted into the agreement table. :|
Comment #20
mradcliffeSo destinations into a database table aren't supported without migrate_plus? Ugh, so frustrating that it's not documented anywhere and not really obvious given all the other destination plugins. Guess I need to implement my own thing similar to #2769979: Add migrate destination for sql table.
Comment #21
mradcliffe- Removes debug code
- todo: d6 fixture and test to make sure that the optional agreement type gets set where it doesn't exist in d6.
Comment #22
mradcliffeAdds drupal 6 migration. I split the agreement migration into d6/d7 versions because I needed to explicitly add a config. option to the plugin to determine the version. Tags don't seem to be able to be used for this without an extraordinary amount of extra code.
I'm not sure if it's worth it to add a migration of the agreement settings in drupal 6 and I think that could be a follow-up.
Comment #23
mradcliffeLost the interdiff locally so recreated using interdiff tools.
Comment #25
mradcliffeUpdating status and title.
I think that this is good enough for an alpha, but I'll leave this open for expanded unit test coverage since most of the tests are functional.
Comment #26
mradcliffeAdds unit tests for some of the complex logic inside AgreementHandler class. There are still some coverage gaps on the class, but the main complexity should be tested.
Comment #27
mradcliffeNot sure why I needed request time defined, but it probably failed at some point in time when I first started this months ago.
Comment #28
mradcliffeI'm satisfied. Patch is only adding test coverage so that's the last thing I'm going to do for this issue. Additional coverage can probably be added later as bugs are reported and fixed.
Comment #30
mradcliffe