From 20ee99ff6ab8248dcbdfd15d5051b44f3d8b7a6c Mon Sep 17 00:00:00 2001
From: alex <alex@cloudware.io>
Date: Sat, 22 Feb 2020 11:44:26 +0100
Subject: [PATCH] Update .php handling to keep <?php at the top

Looking at https://www.php.net/manual/en/language.basic-syntax.phptags.php
there are just too many variations and I'm not convinced the added
complexity to support all of them would be worth it here.

Thus, support only the case where closing tag "?>" is either omitted
or on a separate line. A workaround for all other cases is to add
the license header manually.

Can always revisit this later.
---
 main.go                    | 3 ++-
 testdata/expected/file.php | 4 ++--
 testdata/initial/file.php  | 4 +++-
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/main.go b/main.go
index e753753..13a0108 100644
--- a/main.go
+++ b/main.go
@@ -222,7 +222,7 @@ func licenseHeader(path string, tmpl *template.Template, data *copyrightData) ([
 	case ".html", ".xml":
 		lic, err = prefix(tmpl, data, "<!--", " ", "-->")
 	case ".php":
-		lic, err = prefix(tmpl, data, "<?php", "// ", "?>")
+		lic, err = prefix(tmpl, data, "", "// ", "")
 	case ".ml", ".mli", ".mll", ".mly":
 		lic, err = prefix(tmpl, data, "(**", "   ", "*)")
 	}
@@ -242,6 +242,7 @@ var head = []string{
 	"<!doctype",                // HTML doctype
 	"# encoding:",              // Ruby encoding
 	"# frozen_string_literal:", // Ruby interpreter instruction
+	"<?php",                    // PHP opening tag
 }
 
 func hashBang(b []byte) []byte {
diff --git a/testdata/expected/file.php b/testdata/expected/file.php
index 9205707..f3b1b2b 100644
--- a/testdata/expected/file.php
+++ b/testdata/expected/file.php
@@ -12,6 +12,6 @@
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
-?>
 
-<?php phpinfo(); ?>
+namespace App;
+phpinfo();
diff --git a/testdata/initial/file.php b/testdata/initial/file.php
index 147cebc..868b97d 100644
--- a/testdata/initial/file.php
+++ b/testdata/initial/file.php
@@ -1 +1,3 @@
-<?php phpinfo(); ?>
+<?php
+namespace App;
+phpinfo();