L10n problems with send.php (SMS message for Fennec)

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
6 years ago

People

(Reporter: pascalc, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

9 years ago
The form sending a SMS has several problems that make it difficult to scale for locales:

1/ The message sent is hardcoded in the file:
$si->addMessage ($phone, "Download Mozilla Fennec at http://getfennec.com/");

it should be a variable with a fallback to English so as to be possible to translate the message, like:

if (!isset($SMStext) OR empty($SMStext))
{
  $SMStext = "Download Mozilla Fennec at http://getfennec.com/";
}
$si->addMessage ($phone, $SMStext);

2/ the text (fennec instead of firefox) and url (not working) should probably be updated

3/ send.php should be located in the includes/messagemedia/ so as to avoid duplicating that file for all locales. Internal urls used for redirects via header(Location:) should be updated to use the full url path with the locale code like:

header('Location: http://www.mozilla.com/'.$lang.'/mobile/index.html?status=fail');

(Since send.php does not include our template file, the $lang parameter should be sent via the form variables or extracted from the calling url)
(Reporter)

Comment 1

9 years ago
I have implemented 1/ and 3/ in r51943 on a fork of send.php (sendl10n.php) that I am using for locales, would be probably good to incorporate these change to send.php and maintain only one version of the file though.
(Reporter)

Updated

9 years ago
Blocks: 522973
Depends on: 507202

Comment 2

9 years ago
Pascal,
Waiting on multi-local build URL to replace the stand-in URL in the SMS. Also the SMS copy "Select the link to download Firefox to your Nokia N900" will be localized in the multi-locale build languages. We should have this content end of day tomorrow or Friday.
Pascal, your version looks good.  I pointed the SMS form in en-US to use your version.  At the moment, that is this page,
https://www-trunk.stage.mozilla.com/en-US/mobile/index-later

I'll also use it for bug 526847.  Thanks for doing that.

I also updated the SMS message text per comment #2.

r55230

r55233 removes en-US/mobile/send.php
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Component: www.mozilla.org/firefox → www.mozilla.org
Product: Websites → Websites
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.