Typo in bulk generate text 'a a' and fix the translations to fit this new string.

Takafumi - June 24, 2008 - 15:32
Project:Pathauto
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

I found typo in pathauto.admin.inc. That will be also in 5.x version.

'#description' => t('Maximum number of objects of a given type which should be aliased during a a bulk update. The default is 50 and the recommended number depends on the speed of your server. If bulk updates "time out" or result in a "white screen" then reduce the number.'),

#1

greggles - June 24, 2008 - 16:03
Status:active» needs review

Ah, bummer.

Thanks takafumi for finding this!

AttachmentSize
274328_typo_a_a_5x.patch 1.18 KB

#2

greggles - June 24, 2008 - 16:09
Title:Typo in pathauto.admin.inc» Typo in bulk generate text 'a a'
Status:needs review» active

Fixed in 5.x, 6.x-1, HEAD (6.x-2).

Thanks again takafumi.

Marking this active again because all the translations that we just had people add are broken for that string. Should we fix them?

#3

Takafumi - June 24, 2008 - 16:35

Thanks for fixing.

+1 to fix.

#4

greggles - June 24, 2008 - 16:56
Title:Typo in bulk generate text 'a a'» Typo in bulk generate text 'a a' and fix the translations to fit this new string.
Status:active» fixed

With a little help from sed (sed -i 's/during a a /during a /' *.po) I was able to fix all the translations.

With a little help from Freso, I was able to actually commit the change to HEAD as well.

Thanks, Freso and Takafumi!

#5

Anonymous (not verified) - July 8, 2008 - 17:03
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.