? Language Change ? Pending

User tests: Successful: Unsuccessful:

avatar sakiss
sakiss
20 Sep 2021

Summary of Changes

The PR allows the use of images in the finder's results.

A new setting was added named "Result Image" both in the com_finder's (aka Smart Search) configuration and it's respective "search" menu item.

Testing Instructions

  1. Go to the "Smart Search" component, press the "Clear Index" button and then the "Index".
  2. Press the "Options" button to load the configuration and enable the "Result Image" setting.
  3. Use the smart search in the site's front-end and check the returned articles that have "Intro Image" set.

Actual result BEFORE applying this Pull Request

result_w_images

Expected result AFTER applying this Pull Request

result_images

Sidenotes

Atm only the com_content finder plugin uses the images API function. Support for the rest plugins will be added in a next PR.

There are 3 new settings both in Smart Search Configuration and in Smart Search results menu item:
result_images_params

  1. Result Image > Shows or hides the image in the results.
  2. Image Class > Adds a class to the figure html element. The same that happens with the "Intro Image Class" in com_content.
  3. Linked Image > Creates an anchor for the image. The same that happens with the "Linked Intro Image" in com_content.

Documentation Changes Required

DKN

@ceford @mateoAdi @universewrld it would be great if you could test that again, as i had to close the other PR for that feature.

b168bfb 20 Sep 2021 avatar sakiss CS
38f868d 20 Sep 2021 avatar sakiss Tabs
avatar sakiss sakiss - open - 20 Sep 2021
avatar sakiss sakiss - change - 20 Sep 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Sep 2021
Category Administration com_finder Language & Strings Front End Plugins
avatar sakiss sakiss - change - 20 Sep 2021
The description was changed
avatar sakiss sakiss - edited - 20 Sep 2021
avatar sakiss sakiss - change - 21 Sep 2021
The description was changed
avatar sakiss sakiss - edited - 21 Sep 2021
avatar sakiss sakiss - change - 22 Sep 2021
Labels Added: Language Change ?
avatar Fedik
Fedik - comment - 23 Sep 2021

@sakiss please remove all your changes from administrator/components/com_finder/src/Indexer/Result.php,
Rest is good to go.

avatar mateoAdi mateoAdi - test_item - 30 Sep 2021 - Tested successfully
avatar mateoAdi
mateoAdi - comment - 30 Sep 2021

I have tested this item successfully on a22bd51


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35612.

avatar sakiss
sakiss - comment - 7 Oct 2021

@Fedik Done. Plz remove the "Change Requested" label.
Let's hope that this gets 1 more test and finally got merged.

avatar RickR2H
RickR2H - comment - 10 Oct 2021

I have tested this item successfully on a22bd51

Nice addition to the result page!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35612.

avatar RickR2H RickR2H - test_item - 10 Oct 2021 - Tested successfully
avatar RickR2H RickR2H - change - 10 Oct 2021
Status Pending Ready to Commit
avatar RickR2H
RickR2H - comment - 10 Oct 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35612.

avatar sakiss
sakiss - comment - 22 Oct 2021

RTC?

avatar RickR2H
RickR2H - comment - 22 Oct 2021

Hey Sakis, RTC means Ready To Commit. Next it is up to the release lead to decide if it's a good addition to the platform. If so, they will merge the PR. @bembelimen could you take a look at this PR?

avatar sakiss
sakiss - comment - 22 Oct 2021

@RickR2H the question goes to the release leader

avatar RickR2H
RickR2H - comment - 22 Oct 2021

RL is tagged and PR has been noted!

avatar bembelimen bembelimen - change - 24 Oct 2021
Labels Added: ?
avatar bembelimen bembelimen - change - 24 Oct 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-10-24 19:36:13
Closed_By bembelimen
avatar bembelimen bembelimen - close - 24 Oct 2021
avatar bembelimen bembelimen - merge - 24 Oct 2021
avatar bembelimen
bembelimen - comment - 24 Oct 2021

Thx

avatar sakiss
sakiss - comment - 25 Oct 2021

Thanks!

avatar brianteeman
brianteeman - comment - 1 Apr 2022

Can someone please reconfirm that this functionality works as I cant get it show any images at all

Add a Comment

Login with GitHub to post a comment