I've found a small typo in the declaration of views tables, which means that no relationships work.

Patch to follow.

CommentFileSizeAuthor
#3 storm-views-arguments.patch10.64 KBjuliangb
#1 pm-views-relationships.patch11.58 KBjuliangb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Active » Needs review
FileSize
pm-views-relationships.patch11.58 KB

Patch attached.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Title: Views relationships down't work » Views relationships don't work
Project: Drupal PM (Project Management) » Storm
Component: Code » Storm.module
Status: Needs review » Patch (to be ported)

Committed to PM 7.x-1.x.

This will need to be fixed in Storm 6.x too.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Patch (to be ported) » Needs review
FileSize
storm-views-arguments.patch10.64 KB

Here's a patch to review.

The changed behaviour is only visible when creating a view based on Storm content types, and adding an argument.

Status: Needs review » Needs work

The last submitted patch, storm-views-arguments.patch, failed testing.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Version: 7.x-1.x-dev » 6.x-2.x-dev
Status: Needs work » Needs review

Change to correct branch.

juliangb’s picture

juliangb CreditAttribution: juliangb commented

#3: storm-views-arguments.patch queued for re-testing.