Bug 777449

Summary: Puzzlestring in yasts ldap server module (ldap-server.pot)
Product: [openSUSE] openSUSE 12.2 Reporter: Forgotten User VaeWwCFpKG <forgotten_VaeWwCFpKG>
Component: YaST2Assignee: Peter Varkoly <varkoly>
Status: VERIFIED FIXED QA Contact: Jiri Srain <jsrain>
Severity: Minor    
Priority: P5 - None CC: ke, mfilka, ralf
Version: Final   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description Forgotten User VaeWwCFpKG 2012-08-27 09:49:07 UTC
User-Agent:       Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20100101 Firefox/14.0.1

The two following string are almost certainly connected together, with some sort of variable in the middle. ("Do you really want to remove " + Some package name + "from the MirrorMode setup?\n")



src/mirrormode-wizard.ycp:119
Do you really want to remove 

src/mirrormode-wizard.ycp:121
from the MirrorMode setup?\n

This is a bad practice, since the structure of the sentence may vary depending on the language. Instead it would be better to create one string with a variable inside (like "Do you really want to remove %s from the MirrorMode setup?"). This way the translator can decide for themselves where to place the variable.

Reproducible: Always
Comment 1 Karl Eichwalder 2012-08-29 15:00:14 UTC
Please, check this and fix it for 12.3 and later.
Comment 2 Peter Varkoly 2013-02-28 11:30:22 UTC
Fixed in github and sortly in stable
Comment 3 Martin Vidner 2013-07-29 11:08:05 UTC
GitHub: https://github.com/yast/yast-ldap-server/pull/2