PHPの関数呼び出しでfalseが返ってきたら処理はどうなりますでしょうか？

Question

WordPressのプラグインの記述なのですけれど、
下記PHPプログラム(抜粋しております)の17行目で
$zipping = $this->mfcf7_zl_multilinefile_create_zip($getuploadfiles[$multitag['name']], $zipped_files);

　としてmfcf7_zl_multilinefile_create_zipを呼び出していますが、
この関数の中で、既にファイルが存在していました場合、31行目の
if (file_exists($destination) && !$overwrite) {

　にて、falseが返ると思います。

　そのfalseが返ってきた場合、17行目以降の処理はどうなりますで
しょうか？

　既にファイルが存在しておりますので、ファイルは上書きされ
ませんものの、18行目の
@chmod($zipped_files, 0444);

　とか、既存ファイルに対して処理されてしまうのではないかと
素人ながら思ってしまいます。

　3行目の
$uniqid = uniqid();

　にて、もしファイル名が重なってしまいました場合、このPHPは
どういう処理をしますでしょうか…

以下、falseを返すまでの一部抜粋

      // Update file name

      $uniqid = uniqid();
      $uploads_dir = path_join( wpcf7_upload_dir( 'dir' ), $this->folder_file );
	   wp_mkdir_p( $dir );

      $uploads_dir = $this->wpcf7_maybe_add_random_dir($uploads_dir);
      $zipped_files = trailingslashit($uploads_dir) . $uniqid . '.zip';

      if ($submission) {
         $multitags = $submission->get_contact_form()->scan_form_tags(array(
            'type' => 'multilinefile',
         ));
         $getuploadfiles = $submission->uploaded_files();		
         foreach ($multitags as $multitag) {
            if (!empty($getuploadfiles[$multitag['name']]) && count($getuploadfiles[$multitag['name']]) >= 1) {
               $zipping = $this->mfcf7_zl_multilinefile_create_zip($getuploadfiles[$multitag['name']], $zipped_files);
               @chmod($zipped_files, 0444);

               $array = explode($this->folder_file, $zipped_files);
               $this->file_path_download = $this->get_folder_file_parent() . $array[1];

            }
         }
      }
   }

   function mfcf7_zl_multilinefile_create_zip($files = array(), $destination = '', $overwrite = false)
{
	//if the zip file already exists and overwrite is false, return false
	if (file_exists($destination) && !$overwrite) {
		return false;
	}

Comments

[CHERRY]

mfcf7 などから推測すると https://wordpress.org/plugins/multiline-files-for-contact-form-7/ でしょうか？

公開されているプラグインなら、隠す意味はないと思うので、出典を明記していただけないでしょうか。

[ghon]

ありがとうございます。
multiline-files-for-contact-form-7
cf7-google-sheets-connector
の組み合わせに、
\cf7-google-sheets-connector\includes\class-gs-service.php
を修正しましたものです。

Answer 1

動作を再現出来るコードでは無いので試してはいませんが、

               $zipping = $this->mfcf7_zl_multilinefile_create_zip($getuploadfiles[$multitag['name']], $zipped_files);
               @chmod($zipped_files, 0444);

を一時的に

//               $zipping = $this->mfcf7_zl_multilinefile_create_zip($getuploadfiles[$multitag['name']], $zipped_files);
             $zipping = false;
             @chmod($zipped_files, 0444);

とでもして動作を確認するのが早いかと思いますよ。

@chmod($zipped_files, 0444);

とか、既存ファイルに対して処理されてしまうのではないかと
素人ながら思ってしまいます。

たぶんそうなると思うので、$zippingがfalseで無い事を確認してから処理をする方が良い可能性が高いと思いますよ。これも提示されているソースからだと判断つきませんが。

Comments

[ghon]

ありがとうございます。
$zipping = false;
を追加しましても、追加前と同じようにプログラムは処理されていくようです

また、3行目の
$uniqid = uniqid();
を固定値にしましたけれど、
7行目に(ここが確認漏れ(質問漏れ))
$uploads_dir = $this->wpcf7_maybe_add_random_dir($uploads_dir);
がありまして、既存フォルダかどうか確認をしておりました。

フォルダ(乱数)/ファイル(乱数).zipという形式でした。
確認不足で申し訳ございません。

[tanat]

解決して良かったです。
詳細に調査するのであれば、
xdebug ブレークポイント
あたりで調べて、任意の場所で処理を中断して変数の状態を確認出来るようにすると捗ります

[ghon]

ありがとうございます。