[#36753] - [5.0] Smart Search: Add debugging features
- Fixed in Code Base
- 29 May 2023
- Medium
- Build: 4.1-dev
- # 36753
- Diff
- Hackwar:4.2-finder-debug
User tests: Successful: Unsuccessful:
Summary of Changes
In order to gain wider adoption from third party developers, we need to provide them with improved development tools. This PR implements a few features, which become visible when the debug mode is enabled.
Display indexed item
First of all we want to be able to see what has been indexed for an item. When you enable debugging, the list of entries in the index will now be linked and clicking on such an entry will give you a view which displays the common data from indexed items, the type specific items, the indexed terms and the associated taxonomies. This allows you to understand if the content was indexed the way you expected it, as well as with the weighing you expected.
Debug indexing of one specific item
Again, when debugging is enabled, a new dropdown button in the index view appears with "Index debugging". This button goes to a new view, where you can select a finder plugin and type in an ID to index and it will then run that plugin for that ID. It will then either return what has been prepared to be indexed by the indexer in the result object or whatever output the plugin generated. This should allow a developer to better write a new finder plugin to index their content.
Testing Instructions
- Install Joomla and provide some sample data. Please note that this PR requires "npm i" to be run again after it has been applied!
- Enable debugging mode in global configuration
- Go to Smart Search and click on an entry in the list to display the data in the index for this item
- Click on "Index > Index debugging" and select a finder plugin from the list.
- Type in an ID, for example "60" in case you are using the testing sample data, and click on "Index"
Notes
- The dropdown button seems to have a styling error when using a modal button as first entry. The spacing is wrong and I couldn't find out why.
- The Javascript is most likely "not optimal". I'm happy for corrections.
- I'm open for better language strings.
- Since 4.1 is in RC, this will have to go into 4.2. Since there is no 4.2-dev branch yet, I'm pointing this towards 4.1-dev for now.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_finder Language & Strings Repository NPM Change JavaScript |
| Labels |
Added:
Language Change
NPM Resource Changed
?
|
||
| Labels |
Added:
?
Removed: ? |
||
| Category | Administration com_finder Language & Strings Repository NPM Change JavaScript | ⇒ | Unit Tests Repository Administration com_finder Language & Strings NPM Change JavaScript Front End com_users |
Hmm, that was a failure...
| Labels |
Added:
?
|
||
Please update the markup of the table to follow the accessibility standards used on every table in the admin
| Category | Administration com_finder Language & Strings Repository NPM Change JavaScript Unit Tests Front End com_users | ⇒ | Administration com_finder Language & Strings Repository NPM Change JavaScript |
| Labels |
Removed:
?
|
||
@Hackwar Do you think you can implement @brianteeman 's suggestions?
| Labels |
Added:
?
PBF
|
||
@brianteeman can you have a look if the changes which I made would be acceptable?
The tables still do not have a caption. This is essential.
I kept the caption short. Fine like so? Can you mark those conversations then as resolved?
I need to review it in the browser. Which I will di in the next day or so
width issues
I kept the caption short. Fine like so? Can you mark those conversations then as resolved?
i do nnot have the access to mark the 3 a11y issues as resolved but they as far as i am concerned
For toolback please use the same icon and code as seen in com_workflows transitions
For toolback please use the same icon and code as seen in com_workflows transitions
I don't know how you got that, but my button already looks like the correct one.
width issues
fixed.
this is what I am seeing
It is just ToolbarHelper::back() and that is returning different code here. Maybe you got some other stuff in your codebase interfering there? That is just not what is in the code and what is being returned on my end. And comparing this with our current 4.2-dev, my version and 4.2-dev agree with each other.
done
| Category | Administration com_finder Language & Strings Repository NPM Change JavaScript | ⇒ | Administration com_finder Language & Strings Repository NPM Change JavaScript Libraries |
| Category | Administration com_finder Language & Strings Repository NPM Change JavaScript Libraries | ⇒ | Administration com_finder Language & Strings Repository NPM Change JavaScript |
| Labels |
Removed:
?
PBF
|
||
I am confused.
I expected that I can select an item in the overview
then click on the debug action
then come to the checker view where the Id and type of the plugin are already known from the list view
and there is no need for input (can be disabled)
As it is now, I would not know how to find the Id (of course it is in the slug, but mybe users don't know that.
This pull requests has been automatically converted to the PSR-12 coding standard.
| Labels |
Added:
?
|
||
| Title |
|
||||||
| Labels |
Added:
PR-4.3-dev
|
||
| Labels |
Removed:
?
|
||
| Labels |
Removed:
?
|
||
Leave ID blank or enter a non-existing ID.
| Labels |
Added:
Conflicting Files
|
||
| Title |
|
||||||
| Category | Administration com_finder Language & Strings Repository NPM Change JavaScript | ⇒ | Administration com_banners com_config com_contact com_content com_fields com_finder |
| Title |
|
||||||
| Labels |
Added:
PR-5.0-dev
PR-4.4-dev
Removed: Language Change Conflicting Files NPM Resource Changed PR-4.3-dev |
||
| Category | Administration com_finder com_banners com_config com_contact com_content com_fields | ⇒ | Administration com_finder Language & Strings Repository NPM Change JavaScript |
| Labels |
Removed:
PR-4.4-dev
|
||
| Labels |
Added:
Language Change
NPM Resource Changed
|
||
| Labels |
Added:
?
|
||
I would like to merge this in to j5 do we already have documentation?
No, I need to write the complete documentation for Smart Search still.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-04-18 12:06:26 |
| Closed_By | ⇒ | Hackwar |
| Status | Closed | ⇒ | New |
| Closed_Date | 2023-04-18 12:06:26 | ⇒ | |
| Closed_By | Hackwar | ⇒ |
| Status | New | ⇒ | Pending |
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-05-29 08:16:23 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
Feature
Removed: ? |
||
Thanks
Please change branch. Thanks.