Reviewed: https://reviews.mahara.org/8586 Committed: https://git.mahara.org/mahara/mahara/commit/da597048a2411d58daa96280999847de4e4b9cf9 Submitter: Robert Lyon (robe...@catalyst.net.nz) Branch: 17.10_STABLE
commit da597048a2411d58daa96280999847de4e4b9cf9 Author: Ghada El-Zoghbi <gh...@catalyst-au.net> Date: Wed Jan 24 10:12:35 2018 +1100 Bug 1745042: set link and icon attribute for socialprofiles When a user shares their socialprofile on their profile page and another user searches for that social profile (using elsaticsearch), the icon attribute is never set. Because social profiles are external to Mahara, we can display the link to users. Sponsored by The Australian National University behatnotneeded Change-Id: I94282bed7904f9d37a50b8e0fd32c5f2d174c704 (cherry picked from commit 7f65b938cfb71a6d4a3d49bcdd26e20f16d3bb43) -- You received this bug notification because you are a member of Mahara Contributors, which is subscribed to Mahara. Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it! https://bugs.launchpad.net/bugs/1745042 Title: Social profile icon causing warning in Elasticsearch display Status in Mahara: Fix Committed Status in Mahara 17.10 series: Fix Committed Status in Mahara 18.04 series: Fix Committed Bug description: Mahara: 17:10.2 (2017092611) OS: Linux 16.04 DB: Postgres Browser: any An warning is displayed (and debug is on) when using Elastisearch when searching on data. Please refer to attached image of warning. We are using the raw_old theme but it's also present in raw. I haven't been able to track down the data itself but I can see that it stems from a social profile not having a relevant icon. And, I can track it down to: /htdocs/search/elasticsearch/type/ElasticsearchType_artefact.php::getRecordDataById - lines 353-356 If the array $icons is empty, the properties: link and icon do not get set. Which leads to this warning in the template: /htdocs/theme/raw_old/plugintype/search/elasticsearch/templates/artefact.tpl on line 15. To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/1745042/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : mahara-contributors@lists.launchpad.net Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp