[packagechooser] Sanitize includes

- build was broken due to AppStream moving around
- unnecessary includes
- change name HAVE_XML -> HAVE_APPDATA for meaning

src/modules/packagechooser/CMakeLists.txt

@@ -15,7 +15,7 @@ option( WITH_APPDATA "Support appdata: items in PackageChooser (requires QtXml)"
 if ( WITH_APPDATA )
     find_package(Qt5 COMPONENTS Xml)
     if ( Qt5Xml_FOUND )
-        add_definitions( -DHAVE_XML )
+        add_definitions( -DHAVE_APPDATA )
         list( APPEND _extra_libraries Qt5::Xml )
         list( APPEND _extra_src ItemAppData.cpp )
     endif()

src/modules/packagechooser/Config.cpp

@@ -9,10 +9,17 @@
 
 #include "Config.h"
 
-#ifdef HAVE_XML
+#ifdef HAVE_APPDATA
 #include "ItemAppData.h"
 #endif
 
+#ifdef HAVE_APPSTREAM
+#include "ItemAppStream.h"
+#include <AppStreamQt/pool.h>
+#include <memory>
+#endif
+
+
 #include "GlobalStorage.h"
 #include "JobQueue.h"
 #include "packages/Globals.h"

src/modules/packagechooser/PackageChooserViewStep.cpp

@@ -13,16 +13,6 @@
 #include "PackageChooserPage.h"
 #include "PackageModel.h"
 
-#ifdef HAVE_XML
-#include "ItemAppData.h"
-#endif
-
-#ifdef HAVE_APPSTREAM
-#include "ItemAppStream.h"
-#include <AppStreamQt/pool.h>
-#include <memory>
-#endif
-
 #include "GlobalStorage.h"
 #include "JobQueue.h"
 #include "locale/TranslatableConfiguration.h"

src/modules/packagechooser/Tests.cpp

@@ -9,7 +9,7 @@
 
 #include "Tests.h"
 
-#ifdef HAVE_XML
+#ifdef HAVE_APPDATA
 #include "ItemAppData.h"
 #endif
 #ifdef HAVE_APPSTREAM