Translate profile checkboxes

neochief - July 16, 2009 - 01:22
Project:Internationalization
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Checkboxes fields are not translating in profile view pages. Their title is inside to '#value' argument, no '#title'.

AttachmentSize
i18nprofile-translate-checkboxes.patch834 bytes

#1

Jose Reyero - August 4, 2009 - 15:35
Status:active» needs work

Yes, but we need to check the field is actually a checkbox. Funny things may happen otherwise.

#2

neochief - August 4, 2009 - 16:24
Status:needs work» needs review

Agree, here's an update.

AttachmentSize
i18nprofile-translate-checkboxes2.patch 995 bytes

#3

hass - August 13, 2009 - 19:06
Status:needs review» needs work

Use only double quotes around the SQL string and single quotes around placeholders, please.

$field_type = db_result(db_query('SELECT type FROM {profile_fields} WHERE name = "%s"', $field));

#4

neochief - August 13, 2009 - 21:24
Status:needs work» needs review

Done.

AttachmentSize
i18nprofile-translate-checkboxes2.patch 995 bytes

#5

hass - August 29, 2009 - 17:23

Only as a question as I haven't tested this... what is about 'checkboxes' or radio and others? Are they also not in #title?

If true - if ($field_type == 'checkbox') { may be best to become if (in_array($field_type, array('checkbox', 'checkboxes')) {.

#6

trupal218 - September 1, 2009 - 14:36

Hello everyone,

I am also experiencing the same problem.
Using Content Profile / Content Taxonomy / Term Tree Widget

Thank you all for your support!

#7

Jose Reyero - November 3, 2009 - 17:53
Status:needs review» fixed

Looks good, committed. Thanks.

#8

hass - November 3, 2009 - 18:11

How about checkboxes?

#9

Jose Reyero - November 5, 2009 - 15:16

Are there any checkboxes on the profile?

#10

System Message - November 19, 2009 - 15:20
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.