openSUSE Build Service

Overview Repositories Revisions Requests Users Attributes Meta

File notes.txt of Package kget5

- metalinkcreator: Investigate whether it's possible to port away from KLocale (kdelibs4support),
  maybe with the help of kcontacts and kconfigwidgets.
  Relevant links:
  https://cgit.kde.org/kget.git/commit/?id=58e62c1c80201bf06fe14dedd0c98256d6be3576
  https://cgit.kde.org/kcontacts.git/commit/?id=87881fd3c5f400ea13b896885b2addb92c795825
  https://phabricator.kde.org/D10446
  https://invent.kde.org/sdk/lokalize/-/merge_requests/6/diffs

Nonsens in KGet::getValidDestUrl(), Zeile 1083:
        if (filename.isEmpty())
            filename = QUrl::toPercentEncoding( srcUrl.toString(), "/" );
        destUrl.adjusted( QUrl::RemoveFilename ); <== hat absolut keine Wirkung!
        destUrl.setPath(destUrl.path() + filename);
(an KDE4 Code orientieren)

Außerdem in KGet::isValidDestDirectory(), 1057:
       if (QFileInfo(QUrl(destDir).adjusted(QUrl::RemoveFilename).toString()).isWritable())
            return (!destDir.isEmpty());
        if (!QFileInfo(QUrl(destDir).adjusted(QUrl::RemoveFilename).toString()).isWritable() && !destDir.isEmpty())
(könnte aber sogar "funktionieren"..., sh. debug.patch, trotzdem fixen!)

und sh. UrlChecker::existingDestination() debug output in kget.patch, mögliches Problem.

kget.patch committen (ohne debug output natürlich), praktisch erledigt
transfer-plugins/kio/transferKio.cpp setNewDestination() ?
history.patch auch (erledigt), und zumindest Hunk#2 von fix-metalink.patch (ist gemacht, aber Hunk#1 nicht vergessen).
Frage: (speziell für kget.patch) soll ich QUrl::toLocalFile() oder QUrl::toString(QUrl::PreferLocalFile) verwenden?
Bei destDirs macht letzteres ja eigtl. keinen Sinn, da sie sowieso nur lokale directories sein können sollten, und wenns an QDir oder QFile übergeben wird sowieso nicht...
Für die 2 Source Sachen könne ich eigentlich beim Original bleiben.

transfermultisegkio.cpp:
@@ -338,7 +338,7 @@ FileModel *TransferMultiSegKio::fileModel()
 {
     if (!m_fileModel)
     {
-        m_fileModel = new FileModel(QList<QUrl>() << m_dest, m_dest.upUrl(), this);
+        m_fileModel = new FileModel(QList<QUrl>() << m_dest, m_dest.adjusted(QUrl::RemoveFilename), this); //WOLFI: porting bug? upUrl()
         connect(m_fileModel, SIGNAL(rename(QUrl,QUrl)), this, SLOT(slotRename(QUrl,QUrl)));
 
         QModelIndex statusIndex = m_fileModel->index(m_dest, FileItem::Status);

passierte gestern bei "Redownload Transfer" (right-click context menu um Hauptfenster):
kget: Source: "https://mail.kde.org/pipermail/release-team/2017-November/010681.html" , dest:  "file:///home/wolfi/Downloads/010681.html" , sugg file:  "010681.html" 
KGet::addTransfer(), dest ist ein QString
Fehler ist scheinbar in KGet::redownloadTransfer() (sollte destDir als local path, nicht URL übergeben), obwohl sollte in addTransfer() wirklich destDir direkt als QUrl zugewiesen werden? (sind scheinbar auch noch andere Ungereimtheiten in addTransfer()...)
Anderes Problem, evtl. tlw. verwandt: bei Redownload (aber auch "addTransfer" via DBUS), kommt eine "Ordner schreibgeschützt" Fehlermeldung, danach ein "Speichern unter" Dialog mit verdoppeltem Dateinamen

KGet::destFileInputDialog( "file:///home/wolfi/Downloads/" <===== ,  "010681.html" ), original "file:///home/wolfi/Downloads/" , now "/home/wolfi/Downloads/"
wird von KGet::addTransfer() aufgerufen, mit übergebenem destDir (also *mit* "file://")
Übrigens, multiseg.patch:
@@ -994,7 +999,7 @@ QUrl KGet::destFileInputDialog(QString destDir, const QString& suggestedFileName
 
     QUrl destUrl = KFileDialog::getSaveUrl(startLocation, QString(), m_mainWindow, i18n("Save As"));
     if (!destUrl.isEmpty()) {
-        Settings::setLastDirectory(destUrl.adjusted(QUrl::RemoveFilename).toString());
+        Settings::setLastDirectory(destUrl.adjusted(QUrl::RemoveFilename).toLocalFile()); qDebug() << "KGet::destFileInputDialog(" << destDir << ", " << suggestedFileName << "), original" << destUrl.adjusted(QUrl::RemoveFilename).toString() << ", now" << destUrl.adjusted(QUrl::RemoveFilename).toLocalFile();
     }
 
     return destUrl;

im CMakeLists.txt sind ein paar source files auskommentiert. Fehler?

transferhistory dialog evtl. ändern so wie in transfersettings.cpp

bool DataSourceFactory::checkLocalFile() needed? (see comments)
gibt momentan einfach nur "success" (als debug output) aus und returnt true. Evtl. der Grund für den "invalid Url" Fehler von KIO::open() (m_dest Datei wird nicht angelegt)
Ist in der Tat so, und gefixt... ;-)

Frage: ist .torrent Datei noch notwendig wenn BTTransfer gestartet wurde?
Nein, aber die anderen Dateien in tmp, ansonsten startet der Download von vorn.

Places

File notes.txt of Package kget5

Places