The default bug view has changed. See this FAQ.

DRAE search plugin is broken

RESOLVED FIXED

Status

Mozilla Localizations
es-ES / Spanish
--
major
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: willyaranda, Assigned: rickiees)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

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).

Comment 1

5 years ago
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.
(Assignee)

Comment 4

5 years ago
(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.
(Assignee)

Comment 5

5 years ago
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.
Attachment #635441 - Flags: review?(milos)

Comment 6

5 years ago
yeah, this should just work.
Yes, great, thanks! I've been very busy :(
(Assignee)

Comment 8

5 years ago
(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 9

5 years ago
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+
(Assignee)

Comment 10

5 years ago
Committed:

http://hg.mozilla.org/l10n-central/es-ES/rev/62d965da47ee
http://hg.mozilla.org/releases/l10n/mozilla-aurora/es-ES/rev/88079b783c0c
http://hg.mozilla.org/releases/l10n/mozilla-beta/es-ES/rev/87928a9ea33b

Thanks, Milos. We'll test these builds and sign-off if everything is OK.
Status: NEW → ASSIGNED
(Assignee)

Comment 11

5 years ago
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?

Comment 12

5 years ago
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.
(Assignee)

Comment 13

5 years ago
(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.

Comment 14

5 years ago
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.
(Assignee)

Comment 15

5 years ago
(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.

Comment 16

5 years ago
That one looks good
The patch is working in latest Nightly and I'm OK with the page found by Ricardo for the <SearchForm>.
(Assignee)

Comment 18

5 years ago
Created attachment 638215 [details] [diff] [review]
Revised patch with a better, alternative SearchForm URL

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+
(Assignee)

Comment 20

5 years ago
Thank you, landed:

http://hg.mozilla.org/l10n-central/es-ES/rev/9b634e82e295
http://hg.mozilla.org/releases/l10n/mozilla-aurora/es-ES/rev/c778bdd3d931
http://hg.mozilla.org/releases/l10n/mozilla-beta/es-ES/rev/1e68bcda5c89
(Assignee)

Comment 21

5 years ago
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.
(Assignee)

Comment 22

5 years ago
I think this can be closed safely now.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.