#595 closed Defect (fixed)
Overloading of language string 'email'
| Reported by: | fuhrmara@… | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | Elgg 1.8.1b |
| Component: | Core | Version: | 1.1 |
| Severity: | minor | Keywords: | |
| Cc: | brettp | Difficulty: | trivial |
Description
currently register.php and the usersettings.php of notifications use the same translation tag (email).
But they have a different meaning (register = email-address, notification = notify-type)
if think, you have to change line 28 in \views\default\notifications\settings
from
<td><?php echo elgg_echo($k); ?>: </td>
to
<td><?php echo elgg_echo('notification:' . $k); ?>: </td>
and add an extra translation-tag in en.php
Change History (9)
comment:1 Changed 4 years ago by marcus
- Priority changed from major to trivial
comment:2 Changed 4 years ago by cash
- Summary changed from missing translation tag to Overloading of language string 'email'
comment:3 Changed 3 years ago by brettp
- Milestone set to Elgg 1.8
- Priority changed from trivial to minor
- Type changed from unconfirmed defect to confirmed defect
comment:4 Changed 3 years ago by brettp
- Difficulty set to trivial
comment:5 Changed 3 years ago by cash
- Milestone changed from Elgg 1.8 to Elgg 1.8.1
comment:6 Changed 20 months ago by cash
- Milestone changed from Elgg 1.8.x to Elgg 1.8.1
- Priority changed from normal to low
comment:7 Changed 20 months ago by Cash Costello
- Resolution set to fixed
- Status changed from new to closed
comment:8 Changed 20 months ago by brettp
- Milestone changed from Elgg 1.8.1 to Elgg 1.8.1b
comment:9 Changed 19 months ago by Cash Costello
Fixes #595 separating email address from email notification in language strings
Changeset: 86547ed43ca5a970bf118e4f3c88fb6733209c45
Note: See
TracTickets for help on using
tickets.
Fixes #595 separating email address from email notification in language strings