We have moved to Github. Please open tickets there.

Opened 4 years ago

Closed 17 months ago

#1351 closed Defect (fixed)

Wrong site_guid management in get_tag

Reported by: fabcol Owned by:
Priority: normal Milestone: Elgg 1.8.x
Component: Core Version: 1.6
Severity: minor Keywords: tags, multisite
Cc: brettp Difficulty:

Description

Usual site_guid management in elgg core functions is based on :
site_guid > 0 -> use site_guid value
site_guid = 0 -> use current site_guid
site_guid < 0 -> all sites

in get_tag function, the site guid management is the following
site_guid < 0 -> use current site_guid

else -> all sites (so you never can go to a specific site)

The site_guid should be corrected to match with the usual site guid management

This leads into pbs when working in, multisite environment

Change History (3)

comment:1 Changed 3 years ago by brettp

  • Keywords tags, multisite added; tags removed
  • Milestone changed from Elgg 1.7 to Elgg 2.0

comment:2 Changed 2 years ago by cash

  • Milestone changed from Elgg 2.0 to Elgg 1.8.x

This looks like a defect where are not being consistent and could be fixed in 1.8.x

comment:3 Changed 17 months ago by cash

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

Looks like this was fixed in Elgg 1.7.x with the introduction of elgg_get_tags()

Note: See TracTickets for help on using tickets.