This is a minor issue, but it's creating unnecessary entries in the logs. Trivial patch attached.

CommentFileSizeAuthor
l10n_client.module.patch515 byteshansfn

Comments

mahfiaz’s picture

mahfiaz commented
Status: Active » Reviewed & tested by the community

Please include this patch, this really is one-word change, but avoids a lot of unnecessary junk getting into logs.

mahfiaz’s picture

mahfiaz commented

If you happen to use devel and stacktrace reporting and pushing to remote server is enabled, then it causes an HTTP error 200 every time you save a translation, which causes alert message from browser. This gets quickly pretty annoying.
Therefore my big thanks to anybody who decides to commit this patch. Would be greatly appreciated.

SebCorbin’s picture

SebCorbin commented
Priority: Normal » Major

#2 is so right, annoying little bug, major PITA.

lsolesen’s picture

lsolesen commented
Priority: Major » Normal

Confirmed, that patch takes care of the problem. However, it is still just a normal bug. Only a notice.

candelas’s picture

candelas commented

yes... i am translating and i get this error all the time... and my collaborations get lost...
please, submit this patch :)
thanks a lot for this excellent module!

SebCorbin’s picture

SebCorbin commented
Status: Reviewed & tested by the community » Fixed

Took co-maintainership and committed :)

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Does this apply to Drupal 6? For backport?

SebCorbin’s picture

SebCorbin commented
Status: Fixed » Closed (fixed)

And backported

mahfiaz’s picture

mahfiaz commented

Yay, this is good news :) Thanks a lot.

candelas’s picture

candelas commented

thanks a lot!