Add comment and fix the broken function.

Review-By: Niels Weber

We need to return all components depending on the run mode,
otherwise we would never be able to resolve possible auto
depend on values, call language changed etc...

1 files changed, 10 insertions, 5 deletions

diff --git a/installerbuilder/libinstaller/packagemanagercore.cpp b/installerbuilder/libinstaller/packagemanagercore.cpp
index 34a7c9564..be2965f72 100644
--- a/installerbuilder/libinstaller/packagemanagercore.cpp
+++ b/installerbuilder/libinstaller/packagemanagercore.cpp
@@ -842,14 +842,19 @@ Component* PackageManagerCore::componentByName(const QString &name) const
     return subComponentByName(this, name);
 }
 
+/*!
+    Returns a list of all available components found during a fetch. Note that depending on the run mode the
+    returned list might have different values. In case of updater mode, components scheduled for an
+    update as well as all possible dependencies are returned.
+*/
 QList<Component*> PackageManagerCore::availableComponents() const
 {
-    QList<Component*> result;
-    foreach (QInstaller::Component *component, d->m_rootComponents) {
-        result.push_back(component);
-        result += component->childComponents(true, AllMode);
-    }
+    if (isUpdater())
+        return d->m_updaterComponents + d->m_updaterComponentsDeps;
 
+    QList<Component*> result = d->m_rootComponents;
+    foreach (QInstaller::Component *component, d->m_rootComponents)
+        result += component->childComponents(true, AllMode);
     return result;
 }