The filter for subscriptions under a user tab does not work (like here: localhost/user/2/notifications/subscription).

I made a quick run at fixing it in the attached patch (one line change). With that patch, it will now filter correctly, but the filter form doesn't hide the "status" or "method" fields after filtering one of them.

You will probably need this patch to see the table (except for user1 since user1 has perms): #1307876: Subscriptions - User tab subscriptions list permissions are a little off

CommentFileSizeAuthor
#3 user-tab-filters-1307910-3.patch4.17 KBwebflo
#1 0001-fixed-user-tab-filter.patch1.45 KBthat0n3guy
0001-filters-work-but-method-status-don-t-hide.patch1.14 KBthat0n3guy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

that0n3guy’s picture

that0n3guy CreditAttribution: that0n3guy commented
FileSize
0001-fixed-user-tab-filter.patch1.45 KB

Got it to work. I just unset the items in the form based on the info in $_SESSION. Worked like a charm.

that0n3guy’s picture

that0n3guy CreditAttribution: that0n3guy commented
Status: Needs work » Needs review

needs review though.

webflo’s picture

webflo CreditAttribution: webflo commented
FileSize
user-tab-filters-1307910-3.patch4.17 KB

We should fix the form. Please review.

webflo’s picture

webflo CreditAttribution: webflo commented
Status: Needs review » Fixed

Fixed. Commit 8a6ea2f on 7.x-1.x.

Status: Fixed » Closed (fixed)

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