Making WordPress.org

Opened 3 years ago

Closed 3 years ago

#5804 closed defect (bug) (fixed)

Translation Memory consistenly errors out on bigger strings.

Reported by: nekojonez's profile NekoJonez Owned by: ocean90's profile ocean90
Milestone: Priority: low
Component: Translate Site & Plugins Keywords: close
Cc:

Description

I don't know at what point the translation memory errors out... No clue how long the string has to be.
But, at a certain string length, the translation memory is 100% to error out.

Take for example this string: https://translate.wordpress.org/projects/wp-themes/customify/nl/formal/?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=6057692&filters%5Btranslation_id%5D=55316092

I mostly use this translation memory to check what changed in a fuzzy string.

Change History (3)

#1 follow-up: @ocean90
3 years ago

  • Keywords close added

The query for this string takes around 13 seconds but will return no results. But since the current timeout is set to 2 seconds you will get the error message instead which still seems to be better than waiting for nothing.

The TM is no real help for larger strings like this since they only used once therefore there's nothing we can do here, beside disabling the TM suggestions if the string has more than x characters.

#2 in reply to: ↑ 1 @NekoJonez
3 years ago

Replying to ocean90:

The query for this string takes around 13 seconds but will return no results. But since the current timeout is set to 2 seconds you will get the error message instead which still seems to be better than waiting for nothing.

The TM is no real help for larger strings like this since they only used once therefore there's nothing we can do here, beside disabling the TM suggestions if the string has more than x characters.

Maybe disabling isn't the best idea... I think it would be better that there is a message that says something in the line of: "The translation memory doesn't support strings from this length?"

Maybe I'm thinking too hard in OCD terms for this, but I think it might cause some little confusion when it's gone.

But that's just my two cents.

#3 @ocean90
3 years ago

  • Owner set to ocean90
  • Resolution set to fixed
  • Status changed from new to closed

In 11093:

Translate: Prevent querying the translation memory for long and too unique strings which are not returning results.

Fixes #5804.

Note: See TracTickets for help on using tickets.