Can't locate string in the sources #168

Kapalı
jstolarek tarafından 4 yıl önce açıldı · 6 yorum
jstolarek 4 yıl önce yorum yaptı
Katkıcı

When I mount an encrypted drive Konqueror prompts me to input password to decrypt it. The window title says "Decrypt Storage Device" but I just can't locate it in the source code with greping. Where does that string come from? I'd like to translate it.

When I mount an encrypted drive Konqueror prompts me to input password to decrypt it. The window title says "Decrypt Storage Device" but I just can't locate it in the source code with greping. Where does that string come from? I'd like to translate it.
SlavekB 4 yıl önce yorum yaptı
Sahibi

There seem to be two problems for this string:

  1. There is not used i18n() => the string is hardcoded and therefore not part of the POT/PO files – see tdeioslave/media/mediamanager/dialog.cpp and tdeioslave/media/mounthelper/dialog.cpp
  2. The string is no longer part of the master branch, but only r14.0.x, because in the master branch it is now called Unlock Storage Device => therefore this string will not be available on the TWTW and it will be necessary to make a pull request in the TGW once it is part of the POT/PO for the r14.0.x branch.
    Maybe we could change the string also in r14.0.x branch, because unlock is more appropriate for the operation that is being performed.
There seem to be two problems for this string: 1. There is not used `i18n()` => the string is hardcoded and therefore not part of the POT/PO files – see [tdeioslave/media/mediamanager/dialog.cpp](src/branch/r14.0.x/tdeioslave/media/mediamanager/dialog.cpp#L26) and [tdeioslave/media/mounthelper/dialog.cpp](src/branch/r14.0.x/tdeioslave/media/mounthelper/dialog.cpp#L26) 2. The string is no longer part of the master branch, but only r14.0.x, because in the master branch it is now called *Unlock Storage Device* => therefore this string will not be available on the TWTW and it will be necessary to make a pull request in the TGW once it is part of the POT/PO for the r14.0.x branch.<br/>Maybe we could change the string also in r14.0.x branch, because *unlock* is more appropriate for the operation that is being performed.
MicheleC 4 yıl önce yorum yaptı
Sahibi

In R14.0.x branch, the lock/unlock operations are referred to as encrypted/decrypted and the status of a disk would be shown as such. changing the string to Unlock may be confusing to the users. The icon file names are also namned accordingly, so we should probably not modify the string in my opinion.

If we want to translate the string, we should probably do it through a PR here. So in my opinion we should not port the change done in PR #169 in R14.0.x

In R14.0.x branch, the lock/unlock operations are referred to as encrypted/decrypted and the status of a disk would be shown as such. changing the string to Unlock may be confusing to the users. The icon file names are also namned accordingly, so we should probably not modify the string in my opinion. If we want to translate the string, we should probably do it through a PR here. So in my opinion we should not port the change done in PR #169 in R14.0.x
SlavekB 4 yıl önce yorum yaptı
Sahibi

In R14.0.x branch, the lock/unlock operations are referred to as encrypted/decrypted and the status of a disk would be shown as such. changing the string to Unlock may be confusing to the users. The icon file names are also namned accordingly, so we should probably not modify the string in my opinion.

If we want to translate the string, we should probably do it through a PR here. So in my opinion we should not port the change done in PR #169 in R14.0.x

In essence, the current encrypt / decrypt naming in R14.0.x is confusing to the users because the encrypted partition is actually locked / unlocked. So it seems more appropriate to fix also other strings that use encrypt / decrypt, even though the methods in the R14.0.x branch are called encrypt / decrypt.

> In R14.0.x branch, the lock/unlock operations are referred to as encrypted/decrypted and the status of a disk would be shown as such. changing the string to Unlock may be confusing to the users. The icon file names are also namned accordingly, so we should probably not modify the string in my opinion. > > If we want to translate the string, we should probably do it through a PR here. So in my opinion we should not port the change done in PR #169 in R14.0.x In essence, the current encrypt / decrypt naming in R14.0.x is confusing to the users because the encrypted partition is actually locked / unlocked. So it seems more appropriate to *fix* also other strings that use encrypt / decrypt, even though the methods in the R14.0.x branch are called encrypt / decrypt.
MicheleC 4 yıl önce yorum yaptı
Sahibi

yes, as we discussed in IRC, that is ok.

What should we do about this issue? Close given #169 solved the orinigal problem?

yes, as we discussed in IRC, that is ok. What should we do about this issue? Close given #169 solved the orinigal problem?
MicheleC 3 yıl önce yorum yaptı
Sahibi

Should we close this issue?

Should we close this issue?
jstolarek 3 yıl önce yorum yaptı
Poster
Katkıcı

Yes.

Yes.
SlavekB 3 yıl önce konusunu kapattı
SlavekB 3 yıl önce R14.0.9 release kilometre taşına ekledi
Bu konuşmaya katılmak için oturum aç.
Kilometre Taşı Yok
Atanan Kişi Yok
3 Katılımcı
Bildirimler
Bitiş Tarihi

Bitiş tarihi atanmadı.

Referans: TDE/tdebase#168
Yükleniyor…
Henüz bir içerik yok.