Redirect-articles in search results should have summaries

VERIFIED FIXED in 1.1

Status

VERIFIED FIXED
10 years ago
9 years ago

People

(Reporter: stephend, Assigned: cilias)

Tracking

unspecified
x86
All

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: sumo_only, URL)

(Reporter)

Description

10 years ago
(Apologies for the verbose bug summary; feel free to tweak.)

This might not be a bug, or, it might just be something with user-configurable weights, but wanted to get it filed just in case.

https://support-stage.mozilla.org/tiki-newsearch.php?locale=en&q=firefox+consumes+a+lot+of&where=all&sa=&filter_lang=1&l=en&en_too=1&lastmodif=0&type=0&author= is a substring search for "firefox consumes a lot of", which is the partial title of a KB article (see below).

1) The real article shows up as the 2nd result -- maybe a weight-configurable setting (bug 460224)
2) The 1st result--"CPU Usage" (https://support-stage.mozilla.org/kb/CPU+usage), itself redirects to "Firefox consumes a lot of CPU+resources" (https://support-stage.mozilla.org/kb/Firefox+consumes+a+lot+of+CPU+resources)

It seems that the 1st result shouldn't even show up, since it just redirects to the 2nd anyway?

Updated

10 years ago
Target Milestone: --- → 1.0.2

Updated

10 years ago
Assignee: nobody → paul.craciunoiu
A good solution is to not index any {REDIRECT} articles that lead to other indexed articles (basically, anything that redirects to another local URL).
(Assignee)

Comment 2

10 years ago
There are cases where an alternate title is good for people who search using different terms. For instance "Hiding bookmarks in the Smart Location Bar" redirects to "Cannot clear the Location bar history", and "Removing Internet Explorer" redirects to "Uninstalling Internet Explorer". We should probably have an article called "Preferences window" that redirects "Options window".

So I think we need a different solution than not indexing articles that redirect. 

Can we pin-point the issue here? Is the problem the order of items? Is it the summary? Or should that be a separate bug? In bug 460224, I intentionally put more weight on the article description and data than the title. We've got two or three bugs that overlap, but don't have clearly defined issues.
(Reporter)

Comment 3

10 years ago
I don't have a clear spec either (just the high-level summary, which is more about the technical integration); just filing things as I see them :-)
(Assignee)

Comment 4

10 years ago
As I see it now, bug 485329 is about what to do with redirects we do not want in search results, and this bug is about what to do with redirects we do want in search results. That sound good?

I have an idea on how to fix the article description problem on redirects:
Include the description text in the article before the redirect tag. I assume that the article will redirect right away, no matter if there is other text in the article or not. I've done this on <https://support-stage.mozilla.org/en-US/kb/test+redirect+with+summary>, but need to see it in search results.
(In reply to comment #4)
How about this solution, then:
a) You take all the articles we want in search results and add a summary before the redirect, if this works
b) I (or someone else) change the indexer to not index any articles that start right off with the {REDIRECT}

This is a good approach since we can always change articles around to make them be included on next reindexing without future code changes. This also requires currently existing articles that we want to exclude to start with the {REDIRECT}. Sound good?
(Assignee)

Comment 6

10 years ago
Love it!
David, do you like that plan?
This way, we don't have to remove any articles, and we can pick and choose which articles would show up in search results. (Assuming the test page works after reindexing)

FWIW, the actual markup on the CPU+usage article is:
{REDIRECT(page=Firefox consumes a lot of CPU resources)/}

The markup for the test page I did is:
This is text that should appear in the search results, but this article should still redirect.
{REDIRECT(page=Using the Flash plugin with Firefox)/}

Updated

10 years ago
No longer blocks: 405028
Target Milestone: 1.0.2 → 1.1

Comment 7

10 years ago
Solution:

* In this particular case ("CPU usage") just remove the redirect because it's redundant and obsolete.
* For redirects we would like to keep, make sure the redirect also includes a summary. Cheng will test this on one redirect article to make sure it works.
(Assignee)

Comment 8

10 years ago
Okay, so it's the same plan, except we still go ahead with removing the articles in bug 485329 comment 12.

We also already have a test article for this, see comment 4.

Comment 9

10 years ago
Sorry, I wasn't reading comment 5 or 6. But the proposed plan still holds; no need to add extra logic to the indexer since we can just remove those superfluous article redirects. Thanks!
(Assignee)

Comment 10

10 years ago
Okay, so let's make this bug about adding summaries to the article redirects we want to keep.
Chris, you need me to do anything on this, should I assign it to you?
(Assignee)

Updated

10 years ago
Assignee: paul.craciunoiu → bmo2008
(Assignee)

Comment 12

10 years ago
<Renaming bug>

Now that reindexing is done, searching for "test redirect"[1] yields <https://support-stage.mozilla.org/en/kb/test+redirect+with+summary> in the search results, which shows a summary.

I'll leave this open, because I need to add a summary to the article redirects we want to keep.

[1]<https://support-stage.mozilla.org/tiki-newsearch.php?locale=en&q=test+redirect&where=all&sa=&filter_lang=1&l=en&en_too=1&lastmodif=0&type=0&author=>
Summary: Substring search for article title yields in its 1st result an article which redirects to the original article → Redirect-articles in search results should have summaries
(Assignee)

Comment 13

10 years ago
Summaries are now added (on prod) to the articles:
Removing Internet Explorer
Upgrading Firefox
Hiding bookmarks in the Smart Location Bar

We should be able to test it after the next re-indexing.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Reporter)

Comment 14

10 years ago
Chris, did you mean "Updating Firefox", rather than "Upgrading Firefox"? ("Upgrading Firefox" doesn't seem to have a direct hit in http://support.mozilla.com/tiki-newsearch.php?where=d&locale=en-US&q=upgrading+firefox&sa=.)

[1] "Removing Internet Explore" - http://support.mozilla.com/tiki-newsearch.php?where=d&locale=en-US&q=removing+internet+explorer&sa=
[2] "Updating Firefox" - http://support.mozilla.com/tiki-newsearch.php?where=d&locale=en-US&q=updating+firefox&sa=
[3] "Hiding bookmarks in the Smart Location Bar" - http://support.mozilla.com/tiki-newsearch.php?where=d&locale=en-US&q=hiding+bookmarks+in+the+smart+location+bar&sa=

I think this looks good -- just need clarification on my question.
(Assignee)

Comment 15

10 years ago
'Upgrading Firefox' redirects to 'Updating Firefox'
(Reporter)

Comment 16

10 years ago
(In reply to comment #15)
> 'Upgrading Firefox' redirects to 'Updating Firefox'

Ah, yes, thanks; I was looking for that in the search results.

http://support.mozilla.com/en-US/kb/Upgrading+Firefox does indeed redirect me to http://support.mozilla.com/en-US/kb/Updating+Firefox.

I think we're good here; verified FIXED.
Status: RESOLVED → VERIFIED
Whiteboard: sumo_only
You need to log in before you can comment on or make changes to this bug.