Laravel Excel でxlsxファイルに画像を設定すると破損

Question

前提・実現したいこと

1. Laravel で xlsxファイルを出力します。
2. 処理内で、xlsx内に画像ファイル(png)を貼り付けして出力。
3. そのxlsxファイルを開くと、破損メッセージが表示されます。（問題点）
4. 「はい」をクリックすると、ファイルは画像含めて、正しく表示されます。

★破損メッセージを出ないようにしたいです。★

発生している問題・エラーメッセージ

xlsxの一部の内容に問題が見つかりました。可能な限り内容を回復しますか？ブックの発行元が信頼できる場合は、[はい]をクリックしてください。

該当のソースコード

use Maatwebsite\Excel\Facades\Excel;
use PhpOffice\PhpSpreadsheet\Spreadsheet;
use PhpOffice\PhpSpreadsheet\Reader\Xlsx as ReaderXlsx;
use PhpOffice\PhpSpreadsheet\Writer\Xlsx as WriterXlsx;
use PhpOffice\PhpSpreadsheet\Cell\Coordinate;
use PhpOffice\PhpSpreadsheet\Worksheet\Drawing;

public function excel()
{
    // テンプレートファイル定義
    $excel_file = storage_path('excel/template.xlsx');

    // テンプレートファイル取得
    $reader = new ReaderXlsx();
    $spreadsheet = $reader->load($excel_file);

    // アクティブシートの取得
    $sheet = $spreadsheet->getActiveSheet();

    // 画像設定
    $imagePath = 'name.png';
    (new Drawing())
     ->setPath($imagePath)
     ->setWorksheet($sheet)
     ->setResizeProportional(false)
     ->setCoordinates(Coordinate::stringFromColumnIndex(4) . 1); //セルD1に設定

    // ダウンロード
    // Xlsxにした場合、画像をPHPで設定したファイルを開くと、破損する
    // Xlsにした場合、画像をPHPで設定したファイルを開くと、破損しない
    $writer = \PhpOffice\PhpSpreadsheet\IOFactory::createWriter($spreadsheet, "Xlsx");
    header('Content-Type: application/vnd.openxmlformats-officedocument.spreadsheetml.sheet');
    header('Content-Disposition: attachment; filename="file.xlsx"');    // ダウンロードファイル名
    $writer->save("php://output");  // PHPのoutputストリーム
}

試したこと

• 出力ファイルを「xls」にすると、破損しませんでした。
• テンプレートの読込み方がいけないのか、出力ファイルの形式指定が足りないのか。。
• 画像ファイルは、普通にExcelで挿入、保存した場合は破損になりません。＝画像は悪くない。

補足情報（FW/ツールのバージョンなど）

Windows7
Laravel5.7
Laravel Excel 3.1
phpspreadsheet 1.6
Excel Office365

もし何か、ご助言いただければありがたいです（。。

Answer 1

以前仕事で、同じ症状に、頭を悩ませたことがあります。

その時は、色々と試みた経緯で、たまたま上手く行く方法が見つかったため（VBAマクロ付きのExcelだったので、それらも利用して解決した）、
時間もなかったこともあり、詳しい調査はさておいたまま、とりあえず解決としてしまったのですが、
同じような現象に困っているのこの質問を見つけ、そういえば私も結局原因とか分かってないな・・と思い、
改めて、色々と試してみました。

---

私の検証した限りの話になりますが、どうやら、きっかけはExcelのシートそのものにあるようで。。。

■「新規シート追加　→　シート内を編集　→　保存（1回目）」

ここまでの状態のシートであれば、画像貼り付けで、破損エラーが表示されないのですが、

■「保存（1回目）　→　シート内を編集　→　保存（2回目以降）」

これだと、破損エラーが表示されるようです。

シート内の編集は、こういう変更すべてです。

・セルへの文字追加・削除
・セルの書式設定
・ページ設定
・印刷範囲

これらを、一度保存済みのシートに対して変更→保存し、そのシートに画像を貼りつけると、破損エラー表示に繋がるようです。

もしかすると、シートがこっそり持っている「編集履歴のような情報」が、SpreadSheetから貼り付けた画像に、なんらかの影響を及ぼして、破損メッセージに繋がっているのかもしれません。
「xlsで保存する破損エラーが表示されない」のは、この「編集履歴のような情報」が破棄されるからかも・・・（xlsxとxlsでは、シートの列・行の上限が大きく異なるので、xlsでは持っていられない情報になり得るため、こういった情報が破棄されること自体は不思議ではないと思います）

ということで、PhpSpreadsheet側の問題だとは思いますが、だからといって「破損メッセージがでるのは仕方ないです」とは言えないのが現実なので。

手間だとは思いますが、テンプレートファイルの「シート」を、まるっと新しく作り直してみて下さい。
それで、この現象自体は解決するはず、です（多分　^^;）

作り直しは、新規シート追加で元のシートを全コピペするような方法でお試しください。（シートの複製ではダメ）
また、新規シートは調整が完全に終了するまで、絶対に保存しないよう気を付けてくださいね・・！

Comments

[val0307]

ありがとうございます！！

テストとして、テンプレートを、Excel新規作成しただけのxlsxファイルを使った結果、
破損エラーが発生しませんでした！

この考えで、テンプレートを見直します（。。

[FKM]

読み込み直後で、データを投入する前なら既に加工したテンプレートでも、レイアウトが崩れないまま画像を貼り付けられました。

したがって
１）画像を貼り付ける
２）データを貼り付ける

この順番を守ればまっさらのスプレッドシートでなくても大丈夫なようですね。

Answer 2

Laravelというより、PhpSpreadsheetの問題っぽいですね。
下記の質問が参考になるのではないでしょうか。

• teratail:phpspreadsheetで画像を含むエクセルファイルを出力したい

Comments

[val0307]

回答ありがとうございます。

参考サイトを確認しました。
ここでは、出力ファイル名の重複が問題のようでした。

今回、私はブラウザからExcelファイルをダウンロードする方式としてます。
この場合、自動的に同一ファイル名があったら、連番が付与されるので、
参考サイトの事象は発生していないようです。

[m.ts10806]

ダウンロードするにも一時ファイルを作るので同じとは思います。
ダウンロードではなくファイル作成のみだとどうなりますか？
ただ私自身が知らないだけかもしれませんが「自動的に同一ファイル名があったら、連番が付与される」がピンと来ていません。

[val0307]

* 以下を追加し、ファイル作成のみ実施しましたが、同じ症状でした。
（同じファイル名がないフォルダに出力）

$writer = new WriterXlsx($spreadsheet);
$writer->save('output.xlsx');

* Chromeでファイルをダウンロード時、ダウンロード先に同じファイル名があると、
自動的に、(1),(2),…と連番が付与されました。

* xlsでファイル作成すると、「ファイル形式と拡張子が一致しません。」エラーとなりました。
$writer->save('output.xls');

　テンプレートはxlsxなので、ファイル作成時にxlsにすると、エラーとなるのは納得ですが、
　ファイルダウンロードだと、xlsはOK。xlsxはエラー　なのが、納得いかないです…

[m.ts10806]

＞$writer->save('output.xlsx');
ファイル名同じになってます。

>Chromeでファイルをダウンロード時、ダウンロード先に同じファイル名があると、
自動的に、(1),(2),…と連番が付与されました。

それはあくまでブラウザの機能で同名ファイルのままだと上書きされてしまうのを防ぐためです（あとWindowsのファイルコピー機能とか）。
問題はファイル生成時のPHP側の問題です。
attachmentのfilenameが固定であるので同じファイル名で作っているのは間違いありません。
ファイル名が重複しないように（time()とか日時入れたり）工夫してみてください。

[val0307]

何度もすみません。。

* 以下を修正し、ファイル名が重複しないようにしてみました。
$writer = new WriterXlsx($spreadsheet);
$writer->save('output'.time().'.xlsx');

　＜2回実行した結果＞
　　output1566883364.xlsx
　　output1566883456.xlsx

　と出力されましたが、開くと、同じエラーが発生しました。

　私の勘違いかもしれないので、気づき点がありましたら、ご教示お願いします（。。

[m.ts10806]

save()前に
ob_end_clean(); 入れても同じですよね？

[m.ts10806]

色々見た感じ、先代のPHPExcelの時代から起きている現象ぽいですね。
https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q12126943287
画像が固定画像で固定場所ならテンプレートに含めてみるとか言うてもなくはないですけど・・・

[val0307]

色々ありがとうございます（。。

* ob_end_clean(); 入れてもダメでした。。
* 画像を動的に設定したかったので、テンプレート埋込み方式は外してました。

xlsでの出力とするか、別の方式を検討するか、もう少し悩んでみます。

[m.ts10806]

xlsxのほうが色々と厳密なのかもしれませんね。

[退会済みユーザー]

「xlsxの一部の内容に問題が見つかりました。可能な限り内容を回復しますか？」このメッセージが出るときは、ファイルにPHPのエラーが文字列で出力されているようなことが多いのですが、
出力されたファイルをテキストエディタで強制的に開いた時に、ファイルの先頭、もしくは末尾にPHPのエラー出力や、HTMLのような記述がないか確認してみてください。

[val0307]

ありがとうございます。

xlsxファイルを、テキスト/バイナリエディタで開きました。
PHPエラー、HTMLのような記述は見つかりませんでした。