Client 2.5.1/Linux: Typo

miwie · February 3, 2019, 5:18pm

Ich hatte das bereits vor längerer Zeit schon mal gemeldet. Damals wurde ich vertröstet, es hätte sich bereits viel im Client geändert und man könnte den betreffenden String mit dem Typo nicht finden.

Nun benutze ich den offiziellen Client 2.5.1 (Debian/Testing) und finde den gleichen Fehler wieder:

Unter Einstellungen / Aktivität findet sich “SynchronisertPfad/Dateiname…”.
Gemeint ist die fehlende Trennung zwischen Aktivität und Pfad sowie der Type in Synchronisert.

rakekniven · February 3, 2019, 5:28pm

Das hat nur bedingt was mit Übersetzung zu tun.
Das sind zwei Strings, welche zusammengepackt werden.

Hier muss jemand mit C++ Skillz rann.

Hast Du einen Verweis auf Dein ursprüngliches Posting?

Gruß

miwie · February 3, 2019, 6:53pm

rakekniven · February 3, 2019, 7:04pm

Den Tippfehler habe ich gerade gefixt.

Beim Source ist das schwieriger.

github.com

nextcloud/desktop/blob/652304e82e62f41dad39d346d3226b12c337fcc4/src/gui/activitywidget.cpp#L182


      
                  activity._status = item->_status;
                  activity._dateTime = QDateTime::fromString(QDateTime::currentDateTime().toString(), Qt::ISODate);
                  activity._message = item->_originalFile;
                  activity._link = folderInstance->accountState()->account()->url();
                  activity._accName = folderInstance->accountState()->account()->displayName();
                  activity._file = item->_file;
                  activity._folder = folder;
          
                  if(item->_status == SyncFileItem::NoStatus || item->_status == SyncFileItem::Success){
                      qCWarning(lcActivity) << "Item " << item->_file << " retrieved successfully.";
                      activity._message.prepend(tr("Synced"));
                      _model->addSyncFileItemToActivityList(activity);
                  } else {
                      qCWarning(lcActivity) << "Item " << item->_file << " retrieved resulted in error " << item->_errorString;
                      activity._subject = item->_errorString;
          
                      // add 'protocol error' to activity list
                      _model->addErrorToActivityList(activity);
                  }
              }
          }

miwie · February 3, 2019, 7:28pm

Würde ein zusätzliches …prepend(" "); vor dem …prepend(tr(“Synced”)); nicht ausreidhen/funktionieren?

rakekniven · February 4, 2019, 2:38pm

ja, denke schon.
Willst Du einen PR einreichen?

miwie · February 4, 2019, 3:40pm

Done. Issue #1068

rakekniven · February 4, 2019, 4:16pm

Hallo,

Du hast ein Issue aufgemacht.
Habe einen Pull Request (PR) erstellt und auf Dein Issue verwiesen und auf das Forum hier.

Jetzt sollte das bald einfließen.

rakekniven · February 4, 2019, 4:57pm

Der PR wurde bereits gemerged.