CiviCRM Community Forums (archive)

Elaborating on issue discussed in Multi-Org: Summary of data segregation failures:

Feb 17 16:19:31 <davej_>   I've had a go at multisite_civicrm_aclGroup for events...
Feb 17 16:20:03 <davej_>   .. seems to work in that it's called and sets correct $currentGroups
Feb 17 16:20:39 <davej_>   - but has no effect, e.g. civicrm/event?reset=1 still shows all events.
Feb 17 16:21:02 <davej_>   - likewise civicrm/event/ical?reset=1&page=1
Feb 17 16:21:19 <dlobo>   davej_: this is in 3.1.2?
Feb 17 16:21:29 <dlobo>   can u post on forum, and we'll check and potentially fix in 3.1.3
Feb 17 16:21:39 <davej_>   Yes, 3.1.2
Feb 17 16:21:40 <dlobo>   seems like  a bug if that is happening
Feb 17 16:21:59 <dlobo>   also that code goes in the multisite module eventually right
Feb 17 16:22:30 <davej_>   dlobo: yes, I put it in multisite.module
Feb 17 16:22:46 <dlobo>   cool, you might also want to attach the code there
Feb 17 16:22:54 <dlobo>   did u commit it?
Feb 17 16:22:57 <dlobo>   into svn?
Feb 17 16:23:04 <dlobo>   not sure if u have svn access
Feb 17 16:23:23 <davej_>   dlobo: no, I meant our local copy.

Attached file civi-multisite.patch shows my changes to our local multisite.module. With this code in place, when I visit civicrm/event?reset=1 on site A the debug statements show:

    * multisite_civicrm_aclGroup: allGroups: Array ( [1] => [2] => [3] => Manchester Civi event test2 [5] => Manchester CiviEvent Test 3 [4] => Manchester test civi event 1 [6] => Rochdale Test CiviEvent )
    * multisite_civicrm_aclGroup: initial currentGroups: Array ( [0] => 5 )
    * multisite_civicrm_aclGroup: final currentGroups: Array ( [0] => 5 )

And on site B:

    * multisite_civicrm_aclGroup: allGroups: Array ( [1] => [2] => [3] => Manchester Civi event test2 [5] => Manchester CiviEvent Test 3 [4] => Manchester test civi event 1 [6] => Rochdale Test CiviEvent )
    * multisite_civicrm_aclGroup: initial currentGroups: Array ( [0] => 6 )
    * multisite_civicrm_aclGroup: final currentGroups: Array ( [0] => 6 )

(Event ids 1 to 4 don't have a created_id, I assume because they were created before the code that sets created_id was added.)

However in both cases, all events are shown (except ids 1 & 2 which have no titles).

So: in each case, $currentGroups already has the correct events at the outset, before my code gets called. My code arrives at the same set of events by other means. But then all events are shown.

civicrm/event/ical?reset=1&page=1 also shows all events.

Dave J

Hi Deepak,

The patch is looking good, you are a star. We'll give it some more rigorous testing but seems to do the trick on the pages I've tried:

civicrm/event?reset=1
civicrm/event/ical?reset=1&page=1
civicrm/event/info?reset=1&id=N

Many thanks for addressing this so quickly. My patch to multisite.module seems to be doing the right thing too, so once we've done more testing, maybe we could get that committed.

Thanks,

Dave J

Just verified that the permission checks are same for both dashboard and manage-event screen.

Difference might be due to the fact that dashboard doesn't list events older than a week:
WHERE  ..civicrm_event.start_date >= DATE_SUB( NOW(), INTERVAL 7 day )
Or if that's not the case lets chat on irc and figure out the problem.