Hi,

hook_enable and hook_disable were removed in Drupal 8, so content should be moved to hook_install and hook_uninstall.

Best regards,

Leandro L. Isotton

CommentFileSizeAuthor
#11 move_code_from-2861169-11.patch1 KBrodrigoac

Comments

lisotton created an issue. See original summary.

mgoncalves’s picture

Assigned: Unassigned » mgoncalves
Status: Active » Needs work
mgoncalves’s picture

Assigned: mgoncalves » Unassigned
rodrigoac’s picture

Assigned: Unassigned » rodrigoac

Work in progress.

rodrigoac’s picture

Assigned: rodrigoac » Unassigned
mgoncalves’s picture

Status: Needs work » Closed (won't fix)

This is a module with Drupal 7 structure yet.

ravyg’s picture

Assigned: Unassigned » ravyg
Status: Closed (won't fix) » Needs work

Hello,
Thank for you interest in advance clock.
Some of the work which was initiated to port this module to drupal-8 is done here.:
https://www.drupal.org/sandbox/ravy/2356977

I am looking forward to any help for speeding up the process. I am not currently available to work on it till mid April as I am working my thesis defense. But I will be happy to add moderators and I have also informed current moderators who are proactively looking forward.

1) Piyuesh Kumar https://www.drupal.org/u/piyuesh23
2) Rahul Shinde: https://www.drupal.org/u/rahulshinde

I am re-opening this issue we will move all code at : https://www.drupal.org/sandbox/ravy/2356977
to 8.x-1.x-dev branch.

mgoncalves’s picture

Hello @ravyg,

Yes, we want help you with this....
Could you add @mgoncalves and @renatog as moderator ?

Thank you

ravyg’s picture

Done! added
Added 2 two modules one we created as initiative for porting advance clock to d8 while training in a code sprint. So it might have some work already done. We cab pull that to d8 dev branch.

ravyg’s picture

Saket https://www.drupal.org/u/saki007ster
will create the branch over the weekend.

rodrigoac’s picture

StatusFileSize
new1 KB

Hi guys, as I see that nobody works on that already, I remove the hook_disable from .install file, as peer it was deprecated.

Patch attached !

rodrigoac’s picture

Assigned: ravyg » Unassigned
Status: Needs work » Needs review
renatog’s picture

Assigned: Unassigned » renatog
Status: Needs review » Reviewed & tested by the community
Issue tags: +ciandt-contrib

Hi people.

Works good for me.

Congrats @rodrigoac.

Regards.

  • RenatoG committed 65e8b08 on 8.x-1.x authored by rodrigoac
    Issue #2861169 by rodrigoac, mgoncalves, ravyg, lisotton, RenatoG: Move...
renatog’s picture

Assigned: renatog » Unassigned
Status: Reviewed & tested by the community » Fixed

Fixed.

Commited in dev branch.

Thanks for your contribution @rodrigoac.

Good Work.

Regards.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.