0xf8/pollymc
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

GH-1020 use plain strings for library URLs

Browse Source 
Because the URLs can contain {}, which are percent encoded in URLs and this breaks variable substitution
This commit is contained in:
Petr Mrázek
2015-05-31 17:51:20 +02:00
parent 84757f485b
commit ff64b6cf1d
3 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
logic/minecraft/OneSixUpdate.cpp
View File

@@ -228,7 +228,7 @@ void OneSixUpdate::jarlibStart()
}
QString raw_storage = lib->storageSuffix();
QString raw_dl = lib->url().toString();
QString raw_dl = lib->url();
auto f = [&](QString storage, QString dl)
{

13
logic/minecraft/RawLibrary.cpp
View File

@@ -157,7 +157,7 @@ QJsonObject RawLibrary::toJson() const
m_base_url != "https://" + URLConstants::AWS_DOWNLOAD_LIBRARIES &&
m_base_url != "https://" + URLConstants::LIBRARY_BASE && !m_base_url.isEmpty())
{
libRoot.insert("url", m_base_url.toString());
libRoot.insert("url", m_base_url);
}
if (isNative())
{
@@ -224,7 +224,7 @@ bool RawLibrary::filesExist(const QDir &base) const
}
return true;
}
QUrl RawLibrary::url() const
QString RawLibrary::url() const
{
if (!m_absolute_url.isEmpty())
{
@@ -236,7 +236,14 @@ QUrl RawLibrary::url() const
return QString("https://" + URLConstants::LIBRARY_BASE) + storageSuffix();
}
return m_base_url.resolved(storageSuffix());
if(m_base_url.endsWith('/'))
{
return m_base_url + storageSuffix();
}
else
{
return m_base_url + QChar('/') + storageSuffix();
}
}
bool RawLibrary::isActive() const

6
logic/minecraft/RawLibrary.h
View File

@@ -86,7 +86,7 @@ public: /* methods */
QString storagePath() const;
/// Set the url base for downloads
void setBaseUrl(const QUrl &base_url)
void setBaseUrl(const QString &base_url)
{
m_base_url = base_url;
}
@@ -127,7 +127,7 @@ public: /* methods */
bool isActive() const;
/// Get the URL to download the library from
QUrl url() const;
QString url() const;
protected: /* data */
/// the basic gradle dependency specifier.
@@ -141,7 +141,7 @@ protected: /* data */
public: /* data */
// TODO: make all of these protected, clean up semantics of implicit vs. explicit values.
/// URL where the file can be downloaded
QUrl m_base_url;
QString m_base_url;
/// DEPRECATED: absolute URL. takes precedence the normal download URL, if defined
QString m_absolute_url;