evert/osb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix rare backwards compatibility issue with optimized directives in object "color"

Browse Source
This commit is contained in:
Kae 2023-07-21 17:52:10 +10:00
parent 643e8b15a1
commit 30dd83aa02
2 changed files with 24 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
source/game/StarObject.cpp
View File

@ -144,13 +144,22 @@ void Object::init(World* world, EntityId entityId, EntityMode mode) {
m_orientationDrawablesCache.reset(); m_orientationDrawablesCache.reset();
// This is stupid and we should only have to deal with the new directives parameter, but blah blah backwards compatibility. // This is stupid and we should only have to deal with the new directives parameter, but blah blah backwards compatibility.
auto colorName = configValue("color", "default").toString(); auto colorName = configValue("color", "default").toString().takeUtf8();
auto colorEnd = colorName.find('?'); auto colorEnd = colorName.find('?');
if (colorEnd != NPos) { if (colorEnd != NPos) {
m_colorDirectives = colorName.substr(colorEnd); size_t suffixBegin = colorName.rfind('?');
String colorDirectives;
std::string colorSuffix = suffixBegin == NPos ? "" : colorName.substr(suffixBegin);
if (colorSuffix.empty() && colorSuffix.rfind("?replace", 0) != 0)
colorDirectives = colorName.substr(colorEnd);
else
colorDirectives = colorName.substr(colorEnd, suffixBegin - colorEnd);
m_colorSuffix = move(colorSuffix);
m_colorDirectives = move(colorDirectives);
} }
else else
m_colorDirectives = ""; m_colorDirectives = m_colorSuffix = "";
m_directives = ""; m_directives = "";
if (auto directives = configValue("")) { if (auto directives = configValue("")) {
@ -1234,11 +1243,21 @@ List<Drawable> Object::orientationDrawables(size_t orientationIndex) const {
String& color = m_imageKeys.find("color")->second; String& color = m_imageKeys.find("color")->second;
String backup = move(color); String backup = move(color);
color = backup.substr(0, backup.find('?')); color = backup.substr(0, backup.find('?'));
// backwards compatibility for this is really annoying, need to append text after the <color> tag to the last directive for a rare use-case
auto& image = imagePath.utf8();
size_t suffix = NPos;
if (!m_colorSuffix.empty() && (suffix = image.rfind("<color>")) != NPos)
imagePart.image = String(image.substr(0, (suffix += 7))).replaceTags(m_imageKeys, true, "default");
else
imagePart.image = imagePath.replaceTags(m_imageKeys, true, "default"); imagePart.image = imagePath.replaceTags(m_imageKeys, true, "default");
color = move(backup); color = move(backup);
imagePart.image.directives = layer.imagePart().image.directives; imagePart.image.directives = layer.imagePart().image.directives;
imagePart.addDirectives(m_colorDirectives); imagePart.addDirectives(m_colorDirectives);
if (suffix != NPos)
imagePart.addDirectives(m_colorSuffix + String(image.substr(suffix)).replaceTags(m_imageKeys, true, "default"));
} }
else { else {
imagePart.image = imagePath.replaceTags(m_imageKeys, true, "default"); imagePart.image = imagePath.replaceTags(m_imageKeys, true, "default");

1
source/game/StarObject.hpp
View File

@ -223,6 +223,7 @@ private:
Directives m_directives; Directives m_directives;
Directives m_colorDirectives; Directives m_colorDirectives;
String m_colorSuffix;
Maybe<PeriodicFunction<float>> m_lightFlickering; Maybe<PeriodicFunction<float>> m_lightFlickering;