Hi, I cannot upgrade to beta 2:

Drupal\Core\Extension\InfoParserException: Missing required keys (type, name) in temporary://update-extraction-.../viewsreference/modules/viewsreference_options/viewsreference_options.info.yml in Drupal\Core\Extension\InfoParserDynamic->parse() (line 29 of /web/core/lib/Drupal/Core/Extension/InfoParserDynamic.php).

Comments

maxilein created an issue. See original summary.

maxilein’s picture

Title: Cannot upgrade to beta 2 » Cannot upgrade to beta 2 using D8.3.4
seanb’s picture

That submodule was removed in 2.x. You need to uninstall the module before doing the upgrade and clear the cache. Thank for reporting this, I will add some information to the project page!

NewZeal’s picture

Status: Active » Needs review

I've released rc1 which does not have the submodule in it. That submodule was never meant to be in there in the first place. Your error does however highlight a problem with the Drupal.org packaging script in that it attempted to validate a module that I thought had been disabled.

maxilein’s picture

@seanB Uninstalling is never an option if the module has been in use. All settings would be lost.
@new zeal: thank you for the rc1 I will test it and report back.

NewZeal’s picture

@maxilein, yes sorry about what happened. Hopefully we got on top of it fast enough before it affected too many people. There are unknown numbers of people doing automatic builds using composer and things like that break quite a bit of internet!

seanb’s picture

I believe the module was empty and didn't provide anything to lose right?

Please correct me is I'm wrong! If you have a way to fix the upgrade path I would be happy to fix this.

NewZeal’s picture

rc1 should have fixed it. I didn't meant to include the module in the last push. Obviously that module is redundant now, although there is some merit in providing the advanced options as a separate module. Would you consider putting all your 2.x plugins bar title and argument into a separate module?

seanb’s picture

Do you mean adding back the extra module to 1.x for the plugins and adding the plugin manager to the 1.x version as well? Or is the goal to remove the 2.x branch? I already upgraded some sites, so needing to downgrade to 1.x again would be an issue.

NewZeal’s picture

I created a blank module called viewsreference_options but the 2.x development that you are doing does not require the module. Is that not correct?

seanb’s picture

Ow, I understand what you mean. It's possible to add all the advanced options in a separate module in 2.x. Let's add a separate issue for this.

maxilein’s picture

rc1 fixed it for me. Thank you!

NewZeal’s picture

Yes, it is probably wise to make it a separate module. Otherwise people who want the simple version will end up with a whole lot of stuff they don't want.

seanb’s picture

Created #2896326: Disable extra settings by default to fix this. Instead of adding a separate module, I chose to switch the settings from opt-out to opt-in. This way the extra settings are hidden unless explicitly enabled by the user. This is probably a far better default since I always found the title/argument field confusing for most users as well. The title/argument field are enabled in the upgrade path for users upgrading from 1.x for BC.

Let's continue the discussion in the other issue and let me know what you think.

NewZeal’s picture

Status: Needs review » Fixed

Presume fixed.

NewZeal’s picture

Status: Fixed » Closed (fixed)