Rename getTags to allTags for API consistency

Madour · Jun 15, 2024 · b6dd95c · b6dd95c
1 parent 1bbd931
commit b6dd95c
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/include/LDtkLoader/Entity.hpp b/include/LDtkLoader/Entity.hpp
@@ -52,7 +52,7 @@ namespace ldtk {
         auto getNineSliceBorders() const -> const NineSliceBorders&;
 
         auto hasTag(const std::string& tag) const -> bool;
-        auto getTags() const -> const std::vector<std::string>&;
+        auto allTags() const -> const std::vector<std::string>&;
 
         auto allFields() const -> const std::vector<FieldDef>&;
 

diff --git a/include/LDtkLoader/containers/TagsContainer.hpp b/include/LDtkLoader/containers/TagsContainer.hpp
@@ -13,7 +13,7 @@ namespace ldtk {
     {
     public:
         auto hasTag(const std::string& tag) const -> bool;
-        auto getTags() const -> const std::vector<std::string>&;
+        auto allTags() const -> const std::vector<std::string>&;
 
     protected:
         explicit TagsContainer(const nlohmann::json& j);

diff --git a/src/Entity.cpp b/src/Entity.cpp
@@ -105,9 +105,9 @@ auto Entity::hasTag(const std::string& tag) const -> bool
     return m_definition->hasTag(tag);
 }
 
-auto Entity::getTags() const -> const std::vector<std::string>&
+auto Entity::allTags() const -> const std::vector<std::string>&
 {
-    return m_definition->getTags();
+    return m_definition->allTags();
 }
 
 auto Entity::allFields() const -> const std::vector<FieldDef>&

diff --git a/src/Layer.cpp b/src/Layer.cpp
@@ -61,7 +61,7 @@ Layer::Layer(const nlohmann::json& j, const World* w, const Level* l)
         m_entities.emplace_back(ent, w, this);
         auto& new_ent = m_entities.back();
         m_entities_by_name[new_ent.getName()].emplace_back(new_ent);
-        for (const auto& tag : new_ent.getTags()) {
+        for (const auto& tag : new_ent.allTags()) {
             m_entities_by_tag[tag].emplace_back(new_ent);
         }
     }

diff --git a/src/TagsContainer.cpp b/src/TagsContainer.cpp
@@ -16,7 +16,7 @@ auto TagsContainer::hasTag(const std::string& tag) const -> bool
     return pos != m_tags.end();
 }
 
-auto TagsContainer::getTags() const -> const std::vector<std::string>&
+auto TagsContainer::allTags() const -> const std::vector<std::string>&
 {
     return m_tags;
 }