Skip to:
Content

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#3508 closed defect (bug) (fixed)

Associate two BuddyPress components with the same WordPress page

Reported by: masonjames Owned by:
Milestone: 1.5 Priority: minor
Severity: minor Version: 1.5
Component: Administration Keywords:
Cc:

Description

If a numpty user decides to associate the Activity Streams with the Group page (for example). The Group page will then display Activity (as expected) but the Group component (which is also assigned to the group page) no longer displays at all.

Perhaps we could have some sort of notification in the admin that the user needs to assign a unique page for each component? Or maybe disallow pages that are already in use by another component.

Change History (3)

comment:1 boonebgorges3 years ago

  • Component changed from Settings to Administration
  • Milestone changed from Awaiting Review to 1.5
  • Priority changed from normal to minor
  • Severity changed from normal to minor

This ticket was worthwhile if only so that I could learn the word 'numpty'!

Good catch with this problem. I would say that the behavior itself (one component works while the other one fails) is expected - there's not much we can do about it. As for your two suggestions, disallowing already-selected pages is a nice idea, but there are lots of implementation problems. Ideally, we'd have some sort of javascript slickness on the Dashboard page that removed used pages from the dropdowns, in addition to some server side checks, but that is pretty complicated, especially at this point in the dev cycle.

I think we can fairly easily create a warning message. It should probably go in bp_core_activation_notice(), with other such notices. If anyone wants to work up a patch, it should be pretty straightforward. Otherwise I will try to get to it this week. (Putting in the 1.5 milestone, though I think it's fairly minor and could be bumped if absolutely necessary).

comment:2 boonebgorges3 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [5028]) Adds admin warning when a WP pages is associated with multiple BP components. Fixes #3508

comment:3 masonjames3 years ago

Thanks for adding the warning Boone :)

You can thank @karmatosed for the word 'numpty'. She taught it to me and it's now a regular part of our vocabulary :)

Note: See TracTickets for help on using tickets.