Closed Bug 766515 Opened 13 years ago Closed 12 years ago

DRAE search plugin is broken

Categories

(Mozilla Localizations :: es-ES / Spanish, defect)

Product:
Mozilla Localizations
Component:
es-ES / Spanish
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: willyaranda, Assigned: rpmdisguise-nave)

Details

Attachments

(1 file, 1 obsolete file)

Revised patch with a better, alternative SearchForm URL
868 bytes, patch
Milos
: review+
Details | Diff | Splinter Review
As noticed Benjamín in our NAVE list, there is a change in the DRAE search engine, so our current search plugin is not working. From: > http://buscon.rae.es/draeI/SrvltGUIBusUsual?TIPO_HTML=2&TIPO_BUS=3&LEMA=boria To: > http://lema.rae.es/drae/?val=boria Axel, this patch could land in all trees? (beta/aurora/nightly).
Can we contact RAE to confirm that that's the URL to use? For the implementation, I'd prefer a real patch and a review request to make sure there's no bug in the details. Feel free to request review from Milos, he's doing most of those these days.
Email sent to informatica@rae.es and waiting for response. Yes, I'll attach a patch, of course.
Answer in Spanish from them: ---- Esfectivamente, el cambio es definitivo. Ha habido algunos problemas con la página de redirección cuando se usa la URL anterior, pero ya están resueltos. Un saludo, Fernando Sánchez León Departamento de Tecnología Centro de Estudios de la Real Academia Española ---- So the old URL should redirect to the definition, but with a big delay and not to the new URL. This redirection will be gone in a future. Patch later today.
(In reply to Guillermo López (:willyaranda) from comment #3) > Patch later today. In case you don't have it ready yet, Guillermo, please wait. I'm going to attach a patch to change the searchplugin in both Firefox and Thunderbird.
I wonder if and when this patch lands our current es-ES users will benefit from it without having to do anything on its part. I ask this because, if that's not the case, we should issue a notice in our community sites and instructions for them to manually download and replace it.
Attachment #635441 - Flags: review?(milos)
yeah, this should just work.
Yes, great, thanks! I've been very busy :(
(In reply to Ricardo Palomares from comment #5) > Created attachment 635441 [details] [diff] [review] > Patch to fix DRAE (Spanish normative dictionary) searchplugin for Firefox > and Thunderbird > > I wonder if and when this patch lands our current es-ES users will benefit > from it without having to do anything on its part. I ask this because, if > that's not the case, we should issue a notice in our community sites and > instructions for them to manually download and replace it. ping milos. It would be nice to have this landed, tested and signed-off before next cycle shift. Also, I forgot to say that our plan is to land this patch in beta, aurora and central.
Comment on attachment 635441 [details] [diff] [review] Patch to fix DRAE (Spanish normative dictionary) searchplugin for Firefox and Thunderbird Review of attachment 635441 [details] [diff] [review]: ----------------------------------------------------------------- This looks good. r+ Please land this mentioning this review and bug number in commit message.
Attachment #635441 - Flags: review?(milos) → review+
Still no change in Firefox Beta (although I've got an update) nor in Thunderbird Beta, but I've got it for Thunderbird Aurora and: 1) It works OK. 2) It works with the current user profile, without the user having to do anything special. BTW, what's the purpose of the <SearchForm> tag? I didn't notice and I haven't changed the URL in it. Should I provide a new patch?
The Beta builds will only show the change once you sign-off and the sign-off is taken, by me and Mark, resp. SeachForm is used when you hit enter in the search field without entering any search term, so, yeah, please attach a patch to fix those, too.
(In reply to Axel Hecht [:Pike] from comment #12) > The Beta builds will only show the change once you sign-off and the sign-off > is taken, by me and Mark, resp. OK. I guess the most appropiate course of action then is that we test in Aurora and when it works, we sign-off in both Beta and Aurora. > SeachForm is used when you hit enter in the search field without entering > any search term, so, yeah, please attach a patch to fix those, too. The RAE page is a bit awkard. Most of it are frame-based and features a search form in the header-like. Ping to Spanish guys, I'd go with this URL to display the search form in a more elegant way: http://www.rae.es/rae/Noticias.nsf/Portada2?ReadForm What do you think? Please respond ASAP, so we can provide a patch this Friday.
I think that http://lema.rae.es/drae/ is actually a better page. The page you linked to actually opens a new window if I search in it, that sounds very wrong.
(In reply to Axel Hecht [:Pike] from comment #14) > I think that http://lema.rae.es/drae/ is actually a better page. > > The page you linked to actually opens a new window if I search in it, that > sounds very wrong. Weird thing is I think that's the new style of the website (which is undergoing a redesign, which in turn triggered the problem with the searchplugin). Anyway, don't try to explain why I was unable to find this page before: http://www.rae.es/rae.html which features search boxes for the RAE dictionary and the DPHD, something like a annotated dictionary. The results don't open in a new window.
That one looks good
The patch is working in latest Nightly and I'm OK with the page found by Ricardo for the <SearchForm>.
So, here is the revised patch.
Attachment #635441 - Attachment is obsolete: true
Attachment #638215 - Flags: review?(milos)
Comment on attachment 638215 [details] [diff] [review] Revised patch with a better, alternative SearchForm URL Review of attachment 638215 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. r+. Please go ahead and land.
Attachment #638215 - Flags: review?(milos) → review+
Manually tested copying l10n-central/es-ES/browser/searcplugins/drae.xml to the searchplugins directory of Firefox Beta. Searches are directed to lema.rae.es without going through buscon.rae.es. Hitting [Enter] in the search form with no search terms loads www.rae.es/rae.html, so I've signed off Firefox & Thunderbird in Aurora & Beta channels.
I think this can be closed safely now.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: