回答編集履歴

追加調査

2020/07/11 00:29

投稿

makotokw

スコア24

test CHANGED Viewed

@@ -221,3 +221,91 @@
 }
 ```
+**【根本的な原因】**
+phpspreadsheetの内部で使用している、`zipstream-php`というライブラリにバグがある。
+https://github.com/maennchen/ZipStream-PHP/issues/154
+xlsxファイルを解凍後に再圧縮するのではなく、phpspreadsheetを暫定修正する方法も考えてみた。
+**【phpspreadsheetを暫定的に修正】**
+既にZipファイルに追加されているファイルは複数回追加されないようにチェック。
+`vendor\phpoffice\phpspreadsheet\src\PhpSpreadsheet\Writer\Xlsx.php`
+```php
+public function save($pFilename)
+　　　　　：
+　　　　　：
+《280行目付近》
+            // Add worksheet relationships (drawings, ...)
+            $added_files = array(); // ※追加
+            for ($i = 0; $i < $this->spreadSheet->getSheetCount(); ++$i) {
+                // Add relationships
+                $zip->addFile('xl/worksheets/_rels/sheet' . ($i + 1) . '.xml.rels', $this->getWriterPart('Rels')->writeWorksheetRelationships($this->spreadSheet->getSheet($i), ($i + 1), $this->includeCharts));
+                // Add unparsedLoadedData
+                $sheetCodeName = $this->spreadSheet->getSheet($i)->getCodeName();
+                $unparsedLoadedData = $this->spreadSheet->getUnparsedLoadedData();
+                if (isset($unparsedLoadedData['sheets'][$sheetCodeName]['ctrlProps'])) {
+                    foreach ($unparsedLoadedData['sheets'][$sheetCodeName]['ctrlProps'] as $ctrlProp) {
+                        $zip->addFile($ctrlProp['filePath'], $ctrlProp['content']);
+                    }
+                }
+                if (isset($unparsedLoadedData['sheets'][$sheetCodeName]['printerSettings'])) {
+                    foreach ($unparsedLoadedData['sheets'][$sheetCodeName]['printerSettings'] as $ctrlProp) {
+                        if(!isset($added_files[$ctrlProp['filePath']])) {    // ※追加
+                            $zip->addFile($ctrlProp['filePath'], $ctrlProp['content']);
+                            $added_files[$ctrlProp['filePath']] = $ctrlProp['filePath'];     // ※追加
+                        }    // ※追加
+                    }
+                }
+```

追加調査結果繁栄

2020/07/11 00:29

投稿

makotokw

スコア24

test CHANGED Viewed

@@ -1,17 +1,47 @@
-根本的な原因はわからないのですが、なんとなく状況がわかってきました。
+詳細な状況がわかってきました。
-Excel2019で保存したExcelファイルをテンプレートとして使用するとダメ見たいです。
+テンプレートとなるExcelファイルのシートにプリンタ設定情報が含まている場合、
+そのシートをコピーすると、コピーされた新しいシートにも同じプリンタ設定情報がコピーされます。
+phpspreadsheetで、コピーしたシートのプリンタ設定情報の扱いがバグっている感じです。
+![イメージ説明](20e75c429737802da14f952e9f30257c.jpeg)
+【確認方法】
+1．生成後エラーが出るxlsxファイルの拡張子をzipに変更。
+2．エクスプローラーでzipファイルをダブルクリック。
+3．フォルダ移動。　xl\printerSettings\
+ ※ここのフォルダに同じ名前の printerSettings1.bin がコピーしたシート数分あります。
+**phpspreadsheet自体を修正する技術量はないので、暫定対応で回避する。**
+【暫定対応】
+1．phpspreadsheetで生成されたxlsxファイルをunzipで解凍し、重複ファイルを１つにする。
+2．解凍されたファイルをzipで再圧縮する。　※拡張子は .xlsx で圧縮
 ```php
-<?php
 require '../phpLibs/PhpSpreadsheet/vendor/autoload.php';
@@ -28,19 +58,13 @@
 // Excel2016の環境で、右クリック⇒新規作成⇒Microsoft Excel ワークシート
-// 【生成ファイル】　ＯＫ　！！
 copyTemplateSheet('../tmp/Excel2016_Rclick_NEW.xlsx',
                   '../tmp/Excel2016_Rclick_NEW_generate.xlsx');
-// Excel2016の環境で、右クリック⇒新規作成⇒Microsoft Excel ワークシート
+// Excel2016の環境で、右クリック⇒新規作成⇒Microsoft Excel ワークシート ※新規作成後、ファイルを開いて内容を更新して保存
-// ※新規作成後、ファイルを開いて内容を更新して保存
-// 【生成ファイル】　ＯＫ　！！
 copyTemplateSheet('../tmp/Excel2016_Rclick_NEW_Save.xlsx',
@@ -48,21 +72,23 @@
+// Excel2016の環境で、右クリック⇒新規作成⇒Microsoft Excel ワークシート ※新規作成後、ファイルを開いて内容を更新して保存（プリンタ設定を変更）
+copyTemplateSheet('../tmp/Excel2016_Rclick_NEW_Save2.xlsx',
+                  '../tmp/Excel2016_Rclick_NEW_Save2_generate.xlsx');
 // Excel2019の環境で、右クリック⇒新規作成⇒Microsoft Excel ワークシート
-// 【生成ファイル】　ＯＫ　！！
 copyTemplateSheet('../tmp/Excel2019_Rclick_NEW.xlsx',
                   '../tmp/Excel2019_Rclick_NEW_generate.xlsx');
-// Excel2019の環境で、右クリック⇒新規作成⇒Microsoft Excel ワークシート
+// Excel2019の環境で、右クリック⇒新規作成⇒Microsoft Excel ワークシート ※新規作成後、ファイルを開いて内容を更新して保存
-// ※新規作成後、ファイルを開いて内容を更新して保存
-// 【生成ファイル】　ＮＧ　！！！！
 copyTemplateSheet('../tmp/Excel2019_Rclick_NEW_Save.xlsx',
@@ -76,6 +102,12 @@
 function copyTemplateSheet($load_filename, $save_filename) {
+    $save_folder = realpath(pathinfo($save_filename, PATHINFO_DIRNAME));
+    $save_basename = pathinfo($save_filename, PATHINFO_BASENAME );
     try {
         $reader = new XlsxReader();
@@ -104,6 +136,52 @@
         $writer->save($save_filename);
+        // 解凍先のtmpフォルダ作成
+        if(file_exists($save_filename.'.tmp')) {
+            removeDirectory($save_filename.'.tmp');
+        }
+        mkdir($save_filename.'.tmp', 0777, true);
+        //tmpフォルダにxlsxファイルを解凍　※xlsxファイルはzip圧縮されている？
+        //この段階で、重複しているファイルが１つにまとめられる
+        shell_exec('unzip -o '.$save_filename.' -d '.$save_filename.'.tmp');
+        // 生成されたxlsxファイルを削除
+        unlink($save_filename);
+        // tmpフォルダの内容で再圧縮し、xlsxファイルを作成する
+        $cwd = getcwd();
+        chdir($save_filename.'.tmp');
+        shell_exec('zip -r '.$save_folder.'/'.$save_basename.' *');
+        chdir($cwd);
+        // tmpフォルダを削除
+        removeDirectory($save_filename.'.tmp');
     } catch(Exception $e) {
         die('Error: '.$e->getMessage());
@@ -112,8 +190,34 @@
 }
+function removeDirectory($directory) {
+    $result = true;
+    foreach(new \RecursiveIteratorIterator(
+        new \RecursiveDirectoryIterator($directory, \RecursiveDirectoryIterator::SKIP_DOTS),
+        \RecursiveIteratorIterator::CHILD_FIRST
+    ) as $file) {
+        if ($file->isDir()) {
+            $result &= @rmdir($file->getPathname());
+        } else {
+            $result &= @unlink($file->getPathname());
+        }
+    }
+    return $result && @rmdir($directory);
+}
 ```
-ご回答くださった方々、ありがとうございました♪