After speaking with Dieuwe about porting this module to Drupal 8, I am creating this issue to track the work for this. I have generated a patch based on the work I've done so far and attached it here to provide a starting point to help get this module ported to Drupal 8.
It's not a complete working upgrade, but it's starting to work (module installs in Drupal 8 beta2, the node's weight is updated in the taxonomy_index table, etc), but some functionality (Views sort handler, node links) aren't yet working. It also includes a upgrade-info.html file which was auto-generated from Drupal Module Upgrader to create a report of module hooks
I will continue to try to port this module as time permits, starting with the Views plugin for the sort handler. Any input, direction, or comment is greatly appreciated.
-Tobby
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | nodeorder-8-2368845-09.patch | 54.72 KB | jhedstrom |
| #8 | nodeorder-8-2368845-08.patch | 57.4 KB | jhedstrom |
Comments
Comment #2
dieuweDrupal 8 branch now exists and all future patches can be made against that.
Thanks for all your work so far :)
Comment #3
tobby commentedI've included a new patch that applies against the 8.x-1.x branch. I missed a few files the last time I created a patch, so this covers all of those (.yml files, additional directories, etc) and at least lets the module get successfully enabled now.
Comment #4
jhedstromThis patch adds views support and gets menu links working among other small tweaks.
Comment #5
jhedstromThis patch adds:
Comment #6
dieuweCan you confirm that the last patch should be applied to the 8.x branch? Or should each of the last 3 submitted patches be applied sequentially?
Comment #7
jhedstromThe patch in #5 is against the current 8.x branch, no need to apply the others as they are included.
Comment #8
jhedstromHere's a small fix to the admin form. This patch is directly against the 8.x branch.
Comment #9
jhedstromThis resolves a php notice, removes some unused files, and starts towards overriding the term listing view (see https://www.drupal.org/node/2416129).
Patch is against 8.x branch.
Comment #11
dieuweSorry for the late response here, I have committed the latest patch from #9.
I don't have a Drupal 8 instance to test with at the moment, so I'll rely on you guys to give me a status update.
Comment #12
jhedstromThanks @dieuwe! We're using this on an active project, and it seems to be working as intended. Marking this as fixed--if issues crop up, I'll open new ones to track those.