fix: migrate android App_Resources on tns migrate

Author: DimitarTachev

lib/controllers/migrate-controller.ts

@@ -12,6 +12,7 @@ export class MigrateController extends UpdateControllerBase implements IMigrateC
 		protected $platformsDataService: IPlatformsDataService,
 		protected $packageInstallationManager: IPackageInstallationManager,
 		protected $packageManager: IPackageManager,
+		private $androidResourcesMigrationService: IAndroidResourcesMigrationService,
 		private $devicePlatformsConstants: Mobile.IDevicePlatformsConstants,
 		private $logger: ILogger,
 		private $errors: IErrors,
@@ -103,6 +104,8 @@ export class MigrateController extends UpdateControllerBase implements IMigrateC
 			this.$logger.trace(`Error during auto-generated files handling. ${(error && error.message) || error}`);
 		}
 
+		await this.migrateOldAndroidAppResources(projectData);
+
 		try {
 			await this.cleanUpProject(projectData);
 			await this.migrateDependencies(projectData);
@@ -112,6 +115,14 @@ export class MigrateController extends UpdateControllerBase implements IMigrateC
 		}
 	}
 
+	private async migrateOldAndroidAppResources(projectData: IProjectData) {
+		const appResourcesPath = projectData.getAppResourcesDirectoryPath();
+		if (!this.$androidResourcesMigrationService.hasMigrated(appResourcesPath)) {
+			this.$logger.info("Migrate old Android App_Resources structure.");
+			await this.$androidResourcesMigrationService.migrate(appResourcesPath);
+		}
+	}
+
 	public async shouldMigrate({ projectDir }: IProjectDir): Promise<boolean> {
 		const projectData = this.$projectDataService.getProjectData(projectDir);
 
@@ -134,6 +145,10 @@ export class MigrateController extends UpdateControllerBase implements IMigrateC
 			if (!hasDependency && dependency.shouldAddIfMissing) {
 				return true;
 			}
+
+			if (!this.$androidResourcesMigrationService.hasMigrated(projectData.getAppResourcesDirectoryPath())) {
+				return true;
+			}
 		}
 
 		for (const platform in this.$devicePlatformsConstants) {