This module will be available Drupal 8? I can help with is migration.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

toddy’s picture

Assigned: Unassigned » toddy

Yes, I intend to start porting to D8 soon after the first release candidate of Drupal 8.

toddy’s picture

Status: Needs work » Postponed
platinum1’s picture

Any thoughts on the D8 port, now that 8.0.0 is out?

teraflik’s picture

Any thoughts now? I am getting a hang of Drupal, and I think I might take this up.

toddy’s picture

Hi,

I currently lack the time to do a port of weather to Drupal 8, because so many things in Drupal have changed. I've started several times with a port and quickly discovered every time that I'm not able to simply port the module.

As far as I can see, the module would probably need a complete rewrite from scratch. And for this I simply do not have the time.

So, if you're willing to tackle this, please feel free to start a port.

Regards,
Tobias

joseph.olstad’s picture

Hi Toddy and everyone else, great module, for those interested,
there's a utility for helping with porting from previous versions of Drupal to D8 to make the job a lot easier .

The module is called drupalmoduleupgrader
it does quite a bit of the work for you, and provides code comments with api suggestions for stuff it can't do.

it's probably the best place to start at.

With that said, fantastic module, going to try it out now.

talenus’s picture

Hey all,

For a course I am taking I need to port a module from drupal 7 to drupal 8 as an assignment.
Looks like this weather module still needs to be ported? Has any progress been made? Has anyone tried using the "drupalmoduleupgrader"?

toddy’s picture

Title: D8 » Porting module to Drupal 8

Hi,

as mentioned in #2467809-5: Porting module to Drupal 8 , this module is not easy to port to Drupal 8. So I would not advise to use this module as an assignment for a course. It's really difficult to port. Believe me.

The drupalmoduleupgrader also does not help much with porting this module, I've tried.

Anyway, please be my guest if you still want to port the module to D8 -- it would be much appreciated!

Regards,
Tobias

tarasich’s picture

Version: 7.x-2.8 » 8.x-1.x-dev
Assigned: toddy » Unassigned
Priority: Minor » Normal
Status: Postponed » Needs review
FileSize
1.18 MB

Attaching first version of D8 module port.

Features ported already:

  1. User can import/reimport weather places after module installation (previously it was done in hook_install())
  2. User can add system wide displays and attach weather places to them.
  3. For each system wide display - new block generated.
  4. Users can see detailed forecasts for each weather place attached to system wide display

Would be good to have followups for all other features not ported yet, including tests.

ysamoylenko’s picture

Hello @tarasich,

Thanks for your contribution.

I'm going to apply and test your patch, then I'll get back to you.

tarasich’s picture

FileSize
1.18 MB

Fixed bug with weather places not saved properly. Appeared after refactoring ;)
Attaching new patch, lets check this one.

ysamoylenko’s picture

@tarasich, thanks for your update.

I'll check the functionality again.

ysamoylenko’s picture

Status: Needs review » Reviewed & tested by the community

@tarasich, just re-tested the module functionality.

Everything works as expected.

I'm going to commit your changes and create the first module 'alpha' release for D8.

Great work!

  • ysamoylenko committed 0a7817c on 8.x-1.x authored by tarasich
    Issue #2467809 by tarasich, ysamoylenko: Porting module to Drupal 8
    
ysamoylenko’s picture

Status: Reviewed & tested by the community » Fixed

The patch was committed.

Thanks, everyone for your participation in the module development.

joseph.olstad’s picture

nicely done!

Status: Fixed » Closed (fixed)

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