Nyaaori/rockbox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

rbutil: Simplify manual download URL handling.

Browse source 
- Don't construct the URL directly. The ServerInfo already has it, use
  that instead.
- Don't manually construct the output filename. ZipInstaller will use
  the server filename automatically.

Change-Id: Ib44616daf5efc95b238418b6e699ccd56c40b011
This commit is contained in:
Dominik Riebeling 2020-11-15 18:52:10 +01:00
parent 687edc1866
commit 2b2a7b9ae3
1 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
rbutil/rbutilqt/gui/manualwidget.cpp
View file

@ -65,10 +65,7 @@ void ManualWidget::downloadManual(void)
QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) {
return;
}
QString manual = SystemInfo::platformValue(SystemInfo::Manual).toString();
if(manual.isEmpty()) {
manual = "rockbox-" + SystemInfo::platformValue(SystemInfo::BuildserverModel).toString();
}
QString manual = ServerInfo::platformValue(ServerInfo::ManualPdfUrl).toString();
ProgressLoggerGui* logger = new ProgressLoggerGui(this);
logger->show();
@ -81,13 +78,11 @@ void ManualWidget::downloadManual(void)
installer->setUrl(ServerInfo::platformValue(
ServerInfo::ManualPdfUrl, m_platform).toString());
installer->setLogSection("Manual (PDF)");
installer->setTarget("/" + manual + ".pdf");
}
else {
installer->setUrl(ServerInfo::platformValue(
ServerInfo::ManualZipUrl, m_platform).toString());
installer->setLogSection("Manual (HTML)");
installer->setTarget("/" + manual + "-" + "-html.zip");
}
installer->setLogVersion();
installer->setUnzip(false);