Merge pull request #2170 from fabrixxm/issue_1953

[friendica.git] / mod / wall_attach.php

diff --git a/mod/wall_attach.php b/mod/wall_attach.php
index c23efc7869a7181985afdd73c385dd5d43a74c2a..68752a0e1f659082e51ec6df90add56663d5ba1d 100644 (file)
--- a/mod/wall_attach.php
+++ b/mod/wall_attach.php
@@ -5,17 +5,28 @@ require_once('include/datetime.php');
 
 function wall_attach_post(&$a) {
 
+       $r_json = (x($_GET,'response') && $_GET['response']=='json');
+
        if($a->argc > 1) {
                $nick = $a->argv[1];
                $r = q("SELECT `user`.*, `contact`.`id` FROM `user` LEFT JOIN `contact` on `user`.`uid` = `contact`.`uid`  WHERE `user`.`nickname` = '%s' AND `user`.`blocked` = 0 and `contact`.`self` = 1 LIMIT 1",
                        dbesc($nick)
                );
-               if(! count($r))
+               if(! count($r)){
+                       if ($r_json) {
+                            echo json_encode(array('error'=>t('Invalid request.')));
+                            killme();
+                        }
                        return;
+        }
 
-       }
-       else
+       } else {
+               if ($r_json) {
+                    echo json_encode(array('error'=>t('Invalid request.')));
+                    killme();
+                }
                return;
+    }
 
        $can_post  = false;
        $visitor   = 0;
@@ -52,12 +63,20 @@ function wall_attach_post(&$a) {
                }
        }
        if(! $can_post) {
+               if ($r_json) {
+                    echo json_encode(array('error'=>t('Permission denied.')));
+                    killme();
+                }
                notice( t('Permission denied.') . EOL );
                killme();
        }
 
-       if(! x($_FILES,'userfile'))
+       if(! x($_FILES,'userfile')) {
+               if ($r_json) {
+                    echo json_encode(array('error'=>t('Invalid request.')));
+                }
                killme();
+       }
 
        $src      = $_FILES['userfile']['tmp_name'];
        $filename = basename($_FILES['userfile']['name']);
@@ -65,10 +84,32 @@ function wall_attach_post(&$a) {
 
        $maxfilesize = get_config('system','maxfilesize');
 
+       /* Found html code written in text field of form,
+        * when trying to upload a file with filesize
+        * greater than upload_max_filesize. Cause is unknown.
+        * Then Filesize gets <= 0.
+        */
+
+       if($filesize <=0) {
+               $msg = t('Sorry, maybe your upload is bigger than the PHP configuration allows') . EOL .(t('Or - did you try to upload an empty file?'));
+               if ($r_json) {
+                       echo json_encode(array('error'=>$msg));
+               } else {
+                       notice( $msg. EOL );
+               }
+               @unlink($src);
+               killme();
+       }
+
        if(($maxfilesize) && ($filesize > $maxfilesize)) {
-               notice( sprintf(t('File exceeds size limit of %d'), $maxfilesize) . EOL);
+               $msg = sprintf(t('File exceeds size limit of %s'), formatBytes($maxfilesize));
+               if ($r_json) {
+                       echo json_encode(array('error'=>$msg));
+               } else {
+                       echo  $msg. EOL ;
+               }
                @unlink($src);
-               return;
+               killme();
        }
 
        $r = q("select sum(octet_length(data)) as total from attach where uid = %d ",
@@ -78,7 +119,12 @@ function wall_attach_post(&$a) {
        $limit = service_class_fetch($page_owner_uid,'attach_upload_limit');
 
        if(($limit !== false) && (($r[0]['total'] + strlen($imagedata)) > $limit)) {
-               echo upgrade_message(true) . EOL ;
+               $msg = upgrade_message(true);
+               if ($r_json) {
+                       echo json_encode(array('error'=>$msg));
+               } else {
+                       echo  $msg. EOL ;
+               }
                @unlink($src);
                killme();
        }
@@ -86,7 +132,7 @@ function wall_attach_post(&$a) {
 
        $filedata = @file_get_contents($src);
        $mimetype = z_mime_content_type($filename);
-       $hash = random_string();
+       $hash = get_guid(64);
        $created = datetime_convert();
        $r = q("INSERT INTO `attach` ( `uid`, `hash`, `filename`, `filetype`, `filesize`, `data`, `created`, `edited`, `allow_cid`, `allow_gid`,`deny_cid`, `deny_gid` )
                VALUES ( %d, '%s', '%s', '%s', %d, '%s', '%s', '%s', '%s', '%s', '%s', '%s' ) ",
@@ -102,12 +148,17 @@ function wall_attach_post(&$a) {
                dbesc(''),
                dbesc(''),
                dbesc('')
-       );              
+       );
 
        @unlink($src);
 
        if(! $r) {
-               echo ( t('File upload failed.') . EOL);
+               $msg =  t('File upload failed.');
+               if ($r_json) {
+                       echo json_encode(array('error'=>$msg));
+               } else {
+                       echo  $msg. EOL ;
+               }
                killme();
        }
 
@@ -118,17 +169,24 @@ function wall_attach_post(&$a) {
        );
 
        if(! count($r)) {
-               echo ( t('File upload failed.') . EOL);
+               $msg = t('File upload failed.');
+               if ($r_json) {
+                       echo json_encode(array('error'=>$msg));
+               } else {
+                       echo  $msg. EOL ;
+               }
                killme();
        }
 
-       $lf = '<br />';
+       if ($r_json) {
+            echo json_encode(array('ok'=>true));
+            killme();
+        }
 
-       if(local_user() && intval(get_pconfig(local_user(),'system','plaintext')))
-               $lf = "\n";
+       $lf = "\n";
 
        echo  $lf . $lf . '[attachment]' . $r[0]['id'] . '[/attachment]' . $lf;
-       
+
        killme();
        // NOTREACHED
 }