Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update pm-dl and friends with the new XML update format #249

Closed
damz opened this issue Nov 6, 2013 · 2 comments
Closed

Update pm-dl and friends with the new XML update format #249

damz opened this issue Nov 6, 2013 · 2 comments

Comments

@damz
Copy link
Contributor

damz commented Nov 6, 2013

Drush currently identifies the type of project (module, theme, distribution, etc.) using a vocabulary term that we used to use on Drupal.org when on Drupal 6. Something like this:

<term>
  <name>Projects</name>
  <value>Drupal core</value>
</term>

This was never meant to be used this way. It is very brittle (because it relies on the name of a taxonomy term, that is not a primary key). We broke it a few times in the past.

On Drupal 7, we now have different project types (node types) for the different project types. We added a <type> key in the XML:

<type>project_core</type>

The possible values for this key are:

  project_core
  project_distribution
  project_module
  project_theme
  project_theme_engine
  project_translation

They are guaranteed to be stable, even if we reserve the right to add new values in the future.

We are not using vocabulary anymore for this. So the old keys were missing from the XML when the Drupal 7 version went live. Yesterday, we deployed a compatibility fix that adds back the old keys.

Drush needs to be updated (all supported branches, ideally) to use this new format, as we would like to deprecate the old format as soon as possible.

weitzman added a commit that referenced this issue Nov 7, 2013
…at. Please test this so I can backport to 6.x
weitzman added a commit that referenced this issue Nov 7, 2013
…at. Please test this so I can backport to 6.x
@weitzman
Copy link
Member

weitzman commented Nov 7, 2013

Backported to 6.x and 5.x. I expect to release 6.2.0 this weekend.

@damz
Copy link
Contributor Author

damz commented Nov 7, 2013

Thanks @weitzman! 👍

ergonlogic pushed a commit to ergonlogic/drush that referenced this issue Dec 4, 2013
…date format. Please test this so I can backport to 6.x
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants