Bug 1852 - draksound refers to "above list" when the list actually appears below
Summary: draksound refers to "above list" when the list actually appears below
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal enhancement
Target Milestone: ---
Assignee: Thierry Vignaud
QA Contact:
URL:
Whiteboard:
Keywords: Junior_job
Depends on:
Blocks:
 
Reported: 2011-06-19 12:47 CEST by Remco Rijnders
Modified: 2014-04-10 20:47 CEST (History)
2 users (show)

See Also:
Source RPM: drakxtools-13.58-1.mga1.src.rpm
CVE:
Status comment:

Attachments
Proposed fix (751 bytes, patch)
2013-06-23 18:24 CEST, Remco Rijnders 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Description Remco Rijnders 2011-06-19 12:47:33 CEST 
Description of problem:

When choosing a sound driver, the below message is shown:

#. -PO: keep the double empty lines between sections, this is formatted a la LaTeX
#: harddrake/sound.pm:506
#, c-format
msgid ""
"If you really think that you know which driver is the right one for your card\n"
"you can pick one from the above list.\n"
"\n"
"The current driver for your \"%s\" sound card is \"%s\" "
msgstr ""


However, the list presented actually appears below this text.
Thierry Vignaud 2011-10-04 17:38:55 CEST

Keywords: (none) => Junior_job

Thierry Vignaud 2011-10-04 17:39:21 CEST

Severity: normal => enhancement

Comment 1 Marja Van Waes 2012-01-08 18:27:32 CET 
pinging. because nothing happened to this report since more than 3 months ago, and it still has the status NEW or REOPENED

@ Thierry
Please set status to ASSIGNED if you think this bug was assigned correctly. If for work flow reasons you can't do that, then please put OK on the whiteboard instead.

@ Remco

Is this bug still valid?

CC: (none) => marja11

Manuel Hiebel 2012-10-31 20:04:08 CET

Version: 1 => Cauldron

Comment 2 Remco Rijnders 2013-06-23 18:24:24 CEST 
Created attachment 4158 [details]
Proposed fix
Comment 3 Remco Rijnders 2013-06-23 18:25:32 CEST 
Yes, this bug is still valid.

@tv: I'm unsure how to effectuate the proposed patch so all translators are aware of the updated string. Can you help and/or explain the process needed to follow?
Comment 4 Remco Rijnders 2014-04-09 13:21:31 CEST 
Yuri, you seem to have updated the translation catalogs for all languages before. Can you explain the procedure to me? I'd really like to close this bug report :)

CC: (none) => yurchor

Comment 5 Yuri Chornoivan 2014-04-09 14:09:43 CEST 
(In reply to Remco Rijnders from comment #4)
> Yuri, you seem to have updated the translation catalogs for all languages
> before. Can you explain the procedure to me? I'd really like to close this
> bug report :)

There is a wiki page for this. ;)

https://wiki.mageia.org/en/Fixing_strings_in_code_without_breaking_translations

Or better just fix the string in the code and POT template, push and Transifex will do the rest. Do not do "make update_n_merge" in /po dir because tv does not like this. ;)
Comment 6 Mageia Robot 2014-04-10 20:42:20 CEST 
commit 5b21e2b5c4ffd16ce1f830a13ab9625b7468d45c
Author: Remco Rijnders <remco@...>
Date:   Thu Apr 10 20:42:09 2014 +0200

    The list with drivers appears below, really (mga#1852)
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=5b21e2b5c4ffd16ce1f830a13ab9625b7468d45c
Comment 7 Remco Rijnders 2014-04-10 20:47:49 CEST 
Fixed, hopefully :-)

Status: NEW => RESOLVED
Resolution: (none) => FIXED
Note You need to log in before you can comment on or make changes to this bug.