Elasticsearch - display link to related view for artefacts in search results

Bug #1740208 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon

Bug Description

Now that we use the 'doc' type in Elasticsearch 5+/6+ we need to adjust the views_by_artefact_acl_filter() so that it finds/returns the view(s) related to the artefact being displayed in the results

Robert Lyon (robertl-9)
Changed in mahara:
assignee: nobody → Robert Lyon (robertl-9)
status: New → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/8379

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/8379
Committed: https://git.mahara.org/mahara/mahara/commit/73a105bd5b5d72ba2980bbcf0ce686966cafd586
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 73a105bd5b5d72ba2980bbcf0ce686966cafd586
Author: Robert Lyon <email address hidden>
Date: Wed Dec 27 15:09:42 2017 +1300

Bug 1740208: Fetch related view for artefacts we can see

Fix the way we find the related views now that we don't have a 'view'
type to restrict to. We can achieve this via the internal '_id' value
as that is a combination of old type and id values.

Also strip tags except the <span> one from highlighted description code
in case there is bad html/js in there.

behatnotneeded

Change-Id: I962accadb352b52dd72b61844f7fe0bd0d586ce2
Signed-off-by: Robert Lyon <email address hidden>

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.