checking php files recursively

Author: kumaraksh1

lib/DeploymentProvider/Providers/WebAppDeploymentProvider.js

@@ -124,6 +124,7 @@ class WebAppDeploymentProvider extends BaseWebAppDeploymentProvider_1.BaseWebApp
             const releaseZipPath = path_1.default.join(webPackage, 'release.zip');
             // Ignore if the app is not a Linux app or if release.zip does not exist
             if (!this.actionParams.isLinux || !fs_1.default.existsSync(releaseZipPath)) {
+                core.info(`release.zip does not exist or not a Linux app, skipping deletion: ${releaseZipPath}`);
                 return;
             }
             let isPhpApp = yield this.checkIfTheAppIsPhpApp(webPackage);
@@ -134,7 +135,7 @@ class WebAppDeploymentProvider extends BaseWebAppDeploymentProvider_1.BaseWebApp
             // Delete release.zip if it exists
             try {
                 yield fs_1.default.promises.unlink(releaseZipPath);
-                core.debug(`Deleted release.zip`);
+                core.info(`Deleted release.zip`);
             }
             catch (error) {
                 core.debug(`Error while deleting release.zip for Linux PHP app: ${error}`);
@@ -147,14 +148,24 @@ class WebAppDeploymentProvider extends BaseWebAppDeploymentProvider_1.BaseWebApp
                 // Check if the webPackage folder contains a composer.json file
                 const composerFile = 'composer.json';
                 if (fs_1.default.existsSync(path_1.default.join(webPackage, composerFile))) {
+                    core.info(`Detected PHP app by presence of ${composerFile}`);
                     return true;
                 }
                 // Check if the webPackage folder contains a .php file
-                const hasPhpFiles = fs_1.default.readdirSync(webPackage).some(file => file.endsWith('.php'));
+                core.info(`Checking for .php files in the web package directory: ${webPackage}`);
+                const hasPhpFiles = fs_1.default.readdirSync(webPackage, { withFileTypes: true, recursive: true }).some(file => file.isFile() && file.name.endsWith('.php'));
+                // const entries = fs.readdirSync(webPackage, { withFileTypes: true, recursive: true });
+                // return entries.some(entry => entry.isFile() && entry.name.endsWith('.php'));
+                if (hasPhpFiles) {
+                    core.info(`Detected PHP app by presence of .php files`);
+                }
+                else {
+                    core.info(`No .php files found in the web package directory.`);
+                }
                 return hasPhpFiles;
             }
             catch (error) {
-                core.debug(`Error while checking if the app is PHP: ${error}`);
+                core.info(`Error while checking if the app is PHP: ${error}`);
             }
             return false;
         });

src/DeploymentProvider/Providers/WebAppDeploymentProvider.ts

@@ -9,6 +9,7 @@ import { addAnnotation } from 'azure-actions-appservice-rest/Utilities/Annotatio
 
 import fs from 'fs';
 import path from 'path';
+import { dir } from 'console';
 
 export class WebAppDeploymentProvider extends BaseWebAppDeploymentProvider {
 
@@ -90,6 +91,7 @@ export class WebAppDeploymentProvider extends BaseWebAppDeploymentProvider {
 
         // Ignore if the app is not a Linux app or if release.zip does not exist
         if (!this.actionParams.isLinux || !fs.existsSync(releaseZipPath)) {
+            core.info(`release.zip does not exist or not a Linux app, skipping deletion: ${releaseZipPath}`);
             return;
         }
 
@@ -104,7 +106,7 @@ export class WebAppDeploymentProvider extends BaseWebAppDeploymentProvider {
 
         try {
             await fs.promises.unlink(releaseZipPath);
-            core.debug(`Deleted release.zip`);
+            core.info(`Deleted release.zip`);
         } catch (error) {
             core.debug(`Error while deleting release.zip for Linux PHP app: ${error}`);
         }
@@ -116,17 +118,29 @@ export class WebAppDeploymentProvider extends BaseWebAppDeploymentProvider {
             // Check if the webPackage folder contains a composer.json file
             const composerFile = 'composer.json'; 
             if (fs.existsSync(path.join(webPackage, composerFile))) {
+                core.info(`Detected PHP app by presence of ${composerFile}`);
                 return true;
             }
 
             // Check if the webPackage folder contains a .php file
-            const hasPhpFiles = fs.readdirSync(webPackage).some(file => file.endsWith('.php'));
+            core.info(`Checking for .php files in the web package directory: ${webPackage}`);
+            const hasPhpFiles = fs.readdirSync(webPackage, {withFileTypes: true, recursive: true}).some(file => file.isFile() && file.name.endsWith('.php'));
+
+            // const entries = fs.readdirSync(webPackage, { withFileTypes: true, recursive: true });
+            // return entries.some(entry => entry.isFile() && entry.name.endsWith('.php'));
+
+            if (hasPhpFiles) {
+                core.info(`Detected PHP app by presence of .php files`);
+            } else {
+                core.info(`No .php files found in the web package directory.`);
+            }
 
             return hasPhpFiles;
         } catch (error) {
-            core.debug(`Error while checking if the app is PHP: ${error}`);
+            core.info(`Error while checking if the app is PHP: ${error}`);
         }
 
        return false;
     }
+    
 }