Only use config.platform.php, as it's guaranteed to be the required v…

…ersion of PHP
shivammathur · Jul 29, 2023 · 95916a7 · 95916a7
1 parent a5fb328
commit 95916a7
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 17 deletions.
diff --git a/__tests__/utils.test.ts b/__tests__/utils.test.ts
@@ -300,13 +300,6 @@ describe('Utils tests', () => {
     );
     expect(await utils.readPHPVersion()).toBe('7.4.33');
 
-    existsSync
-      .mockReturnValueOnce(false)
-      .mockReturnValueOnce(false)
-      .mockReturnValueOnce(true);
-    readFileSync.mockReturnValue('{ "require": { "php": "^8.2" } }');
-    expect(await utils.readPHPVersion()).toBe('^8.2');
-
     existsSync.mockClear();
     readFileSync.mockClear();
   });

diff --git a/dist/index.js b/dist/index.js
@@ -1387,10 +1387,6 @@ async function readPHPVersion() {
             composerFileContents['config']['platform']['php']) {
             return composerFileContents['config']['platform']['php'];
         }
-        if (composerFileContents['require'] &&
-            composerFileContents['require']['php']) {
-            return composerFileContents['require']['php'];
-        }
     }
     return 'latest';
 }

diff --git a/src/utils.ts b/src/utils.ts
@@ -464,12 +464,6 @@ export async function readPHPVersion(): Promise<string> {
     ) {
       return composerFileContents['config']['platform']['php'];
     }
-    if (
-      composerFileContents['require'] &&
-      composerFileContents['require']['php']
-    ) {
-      return composerFileContents['require']['php'];
-    }
   }
 
   return 'latest';