Project

General

Profile

Actions

Bug #4714

closed

Backend upgrade to CBSubs plan from user profile needs two tries to work on CB 2.0

Added by nant over 10 years ago. Updated about 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
16 September 2014
Due date:
% Done:

100%

Estimated time:

Description

CBSubs 3.0 install on CB 2.0 RC3.

Create a single CBSubs plan.

When I go to an existing user in CB User manager and visit the subscription tab of the user profile I see the radio button to select the plan (and the default lifetime one) and if I select and then Save button on the user management toolbar the user gets saved but the plan is not activate (radio button still shows options).

If I try a second time it works.

Same exact behavior is happening with unsubscribe checkbox and save in backend - needs to be done twice to work.

Actions #1

Updated by nant over 10 years ago

Same behavior with delete checkbox on active plan in user profile.

Actions #2

Updated by nant over 10 years ago

Ok - feedback discusses with Kyle:

The Save button on CB 1.9.1 User Manager actually worked as a Save and CLose button, so this is something that CBSubs was never really designed to do.

But still needs investigating to be addressed at least in next CBSubs release.

Actions #3

Updated by krileon over 10 years ago

This is due to a bug in CBSubs instance caching.

components/com_comprofiler/plugin/user/plug_cbpaidsubscriptions/libraries/cbpaidSubscriptionsMgr.php

The getInstance function is caching the self instance. On redisplay it's reusing that instance (because we don't redirect on redisplay).

Issue can be partially resolve by redirecting on successful apply instead of redisplaying, but in cases of errors you'll see this issue.

Actions #4

Updated by krileon over 10 years ago

  • % Done changed from 0 to 50

Partially fixed by MR !493 to redirect on successful user applies instead of unnecessary redisplay and reparsing of post. This matches XML usage of Apply. The rest of the fix would need to be done in CBSubs it self.

Actions #5

Updated by krileon over 10 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 50 to 100

Ticket opened for CBSubs side of issue #4720

Actions #6

Updated by beat about 10 years ago

  • Priority changed from High to Normal
Actions #7

Updated by beat about 10 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF