Contrib.social

Customer Subscription list view is broken in 10.3

Open on Drupal.org →
Created on 8 August 2024, about 2 months ago
Updated 23 August 2024, about 1 month ago

Problem/Motivation

Since the view is using URL rewrite feature links in D10.3 look like this
user/admin/subscriptions/26
instead of this
/user/1/subscriptions/26

This leads to 404 on a customer subscription page as the user param is unmet.

Furthermore, this wasn't discovered by automated tests meaning that we have insufficient test coverage.

Steps to reproduce

Install this module on D10.3. Create subscription for a user and go to their user page. Then click on Subscriptions tab and on one of the subscriptions.

Proposed resolution

In views.view.commerce_user_subscriptions.yml change

            path: 'user/{{ arguments.uid }}/subscriptions/{{ subscription_id }}'

to

            path: 'user/{{ raw_arguments.uid }}/subscriptions/{{ subscription_id }}'

Remaining tasks

- add test coverage
- change default config
- provide update hook to change config on existing sites

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇳🇴Norway zaporylie

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024