I am porting this module to Drupal 8 as CI&T/Acquia hackathon in Ningbo today.

CommentFileSizeAuthor
#14 move_user-port-to-d8-2693353-14.patch23.65 KBtclark62
#4 porting-to-d8-20160331.patch19.95 KBbalawang
#2 0001-porting-to-d8.patch51.64 KBbalawang

Comments

balawang created an issue. See original summary.

balawang’s picture

balawang commented
StatusFileSize
new 0001-porting-to-d8.patch51.64 KB

Uploading the patch extracted from hackathon (co-worker: kevin, jieyyal, qing) result with
1. ported the module to D8 with exactly the same functionality as D7 module
2. added test case to D8 module
3. passed drupal code standards

How to use the patch
1. use upgrademodule to upgrade version 7.x-1.x-dev
2. then apply the patch attached

darrenmothersele’s picture

darrenmothersele commented

Please can you remove the binary files from the patch.

balawang’s picture

balawang commented
StatusFileSize
new porting-to-d8-20160331.patch19.95 KB

Hi Mothersele,

The new patch (porting-to-d8-20160331.patch) with removed binary file attached.

balawang’s picture

balawang commented
Issue tags: +#Drupal8 #ciandt-contrib
balawang’s picture

balawang commented
Issue tags: -#Drupal8 #ciandt-contrib +#ciandt-contrib

Correct tags.

adammalone’s picture

adammalone
Primary language English
Location San Francisco
commented
Assigned: balawang » Unassigned
Status: Active » Needs review

Unassigning from Bala so the module maintainer can look over and review it.

adammalone’s picture

adammalone
Primary language English
Location San Francisco
commented

@darrenmothersele is there anything we can assist with from our end to help create a new 8.x branch and push this code into it?

adammalone’s picture

adammalone
Primary language English
Location San Francisco
commented
Priority: Normal » Major

Bumping the priority of this to try and get some traction with the patches in the issue queue.

yechaozheng’s picture

yechaozheng commented

@darrenmothersele is there anything we can assist to help create a new 8.x branch and push this code into it?
@typhonius do you have any suggestion about what can we do now to push this code into 8.x?

yechaozheng’s picture

yechaozheng commented

@darrenmothersele
@typhonius

Any update on this issue?

darrenmothersele’s picture

darrenmothersele commented

These patches don't apply.
Please can you re-roll them so they cleanly apply?

darrenmothersele’s picture

darrenmothersele commented
Status: Needs review » Needs work

There are several issues that prevent this patch applying.

We also need to take into account other entities (not just Nodes in Drupal 8).

tclark62’s picture

tclark62 commented
StatusFileSize
new move_user-port-to-d8-2693353-14.patch23.65 KB

I based this patch on the code from @balawang. I don't really have the intention of continuing to work on this, especially since this module appears dead, but I thought I'd at least share it here, since this code helped me write it. It should apply to against the D7 version.

If anyone is still interested, it might provide a starting point for the dev branch as it seems to work as far as the basics go. It changes the ownership of both nodes and comments.