There is now a proper routing system in core which largely replaces the old hook_menu() functionality. We have to update our menu entries to use that system instead of hook_menu().

More information/questions as soon as I've read the documentation and maybe tried a few conversions.

See WSCCI Conversion guide for help on how to convert legacy page callbacks to new-style controllers.

Comments

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey commented
Title: Adapt menu entries to now Routing system » Adapt menu entries to new Routing system
jsacksick’s picture

jsacksick CreditAttribution: jsacksick commented

I started to work a bit on this but I think we should first fix #2044087: Adapt indexes and servers to new core Entity API, because once I started to work on the overview table, I discovered that the search_api_server entity type wasn't defined.
Plus, I got errors due to the call of entity_get_property_info which doesn't exist anymore.
theme_table hasn't been converted to Twig yet, See #1939008: Convert theme_table() to Twig.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick commented
Issue summary: View changes

Added the WSCII conversion guide link.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey commented

As said in my mail, the whole module code is currently not working, so there's no testing at the moment anyways. We'll just have to code the things as best as we can and wait until enough pieces are set to have a module that at least doesn't explode when being enabled (and then, later, would even be functional).

However, thanks a lot for your help!

freblasty’s picture

freblasty CreditAttribution: freblasty commented
Assigned: Unassigned » freblasty
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey commented

Are you still working on this?
I'm sorry, I didn't have any time for this the last two weeks, but I'm now finally getting back into gear. I'll try to be in IRC more, too, if you want to discuss anything.

freblasty’s picture

freblasty CreditAttribution: freblasty commented

I'm still working on this just had some other issues that I needed to take care of before starting on this.

freblasty’s picture

freblasty CreditAttribution: freblasty commented
Issue summary: View changes

Correct the link

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey commented
Issue summary: View changes
Parent issue: » #2044421: [meta] Upgrade to Drupal 8
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey commented
Assigned: freblasty » Unassigned
Status: Active » Fixed

This is long done now.

Status: Fixed » Closed (fixed)

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