X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fnewapplication.php;h=0f819b3499881795e3ce597fd1559846875f7f15;hb=d06bdfa54bab19c04d4e4dd74801df57bcd5a19c;hp=a0e61d288cf9af978b288ac49aa9c5fd6507b4c6;hpb=b14a97f5f95e809c6b5bffa6dab8eba06509a3dc;p=quix0rs-gnu-social.git

diff --git a/actions/newapplication.php b/actions/newapplication.php
index a0e61d288c..0f819b3499 100644
--- a/actions/newapplication.php
+++ b/actions/newapplication.php
@@ -83,7 +83,7 @@ class NewApplicationAction extends OwnerDesignAction
         parent::handle($args);
 
         if ($_SERVER['REQUEST_METHOD'] == 'POST') {
-	    $this->handlePost($args);
+        $this->handlePost($args);
         } else {
             $this->showForm();
         }
@@ -91,36 +91,35 @@ class NewApplicationAction extends OwnerDesignAction
 
     function handlePost($args)
     {
-	// Workaround for PHP returning empty $_POST and $_FILES when POST
+    // Workaround for PHP returning empty $_POST and $_FILES when POST
         // length > post_max_size in php.ini
 
         if (empty($_FILES)
             && empty($_POST)
             && ($_SERVER['CONTENT_LENGTH'] > 0)
-	    ) {
+        ) {
             $msg = _('The server was unable to handle that much POST ' .
-		     'data (%s bytes) due to its current configuration.');
+             'data (%s bytes) due to its current configuration.');
             $this->clientException(sprintf($msg, $_SERVER['CONTENT_LENGTH']));
             return;
         }
 
-	// CSRF protection
-	$token = $this->trimmed('token');
-	if (!$token || $token != common_session_token()) {
-	    $this->clientError(_('There was a problem with your session token.'));
-	    return;
-	}
-
-	$cur = common_current_user();
-
-	if ($this->arg('cancel')) {
-	    common_redirect(common_local_url('apps',
-					     array('nickname' => $cur->nickname)), 303);
-	} elseif ($this->arg('save')) {
-	    $this->trySave();
-	} else {
-	    $this->clientError(_('Unexpected form submission.'));
-	}
+    // CSRF protection
+    $token = $this->trimmed('token');
+    if (!$token || $token != common_session_token()) {
+        $this->clientError(_('There was a problem with your session token.'));
+        return;
+    }
+
+    $cur = common_current_user();
+
+    if ($this->arg('cancel')) {
+        common_redirect(common_local_url('oauthappssettings'), 303);
+    } elseif ($this->arg('save')) {
+        $this->trySave();
+    } else {
+        $this->clientError(_('Unexpected form submission.'));
+    }
     }
 
     function showForm($msg=null)
@@ -147,7 +146,7 @@ class NewApplicationAction extends OwnerDesignAction
 
     function trySave()
     {
-	$name         = $this->trimmed('name');
+        $name         = $this->trimmed('name');
         $description  = $this->trimmed('description');
         $source_url   = $this->trimmed('source_url');
         $organization = $this->trimmed('organization');
@@ -159,6 +158,9 @@ class NewApplicationAction extends OwnerDesignAction
         if (empty($name)) {
              $this->showForm(_('Name is required.'));
              return;
+        } else if ($this->nameExists($name)) {
+            $this->showForm(_('Name already in use. Try another one.'));
+            return;
         } elseif (mb_strlen($name) > 255) {
             $this->showForm(_('Name is too long (max 255 chars).'));
             return;
@@ -200,8 +202,8 @@ class NewApplicationAction extends OwnerDesignAction
         {
             $this->showForm(_('Homepage is not a valid URL.'));
             return;
-        } elseif (empty($callback_url)) {
-            $this->showForm(_('Callback is required.'));
+        } elseif (mb_strlen($callback_url) > 255) {
+            $this->showForm(_('Callback is too long.'));
             return;
         } elseif (strlen($callback_url) > 0
             && !Validate::uri(
@@ -266,48 +268,29 @@ class NewApplicationAction extends OwnerDesignAction
             $app->query('ROLLBACK');
         }
 
-	$this->uploadLogo($app);
+        $app->uploadLogo();
 
         $app->query('COMMIT');
 
-        common_redirect(common_local_url('apps',
-            array('nickname' => $cur->nickname)), 303);
+        common_redirect(common_local_url('oauthappssettings'), 303);
 
     }
 
     /**
-     * Handle an image upload
+     * Does the app name already exist?
      *
-     * Does all the magic for handling an image upload, and crops the
-     * image by default.
+     * Checks the DB to see someone has already registered an app
+     * with the same name.
      *
-     * @return void
+     * @param string $name app name to check
+     *
+     * @return boolean true if the name already exists
      */
 
-    function uploadLogo($app)
+    function nameExists($name)
     {
-        if ($_FILES['app_icon']['error'] ==
-            UPLOAD_ERR_OK) {
-
-	    try {
-		$imagefile = ImageFile::fromUpload('app_icon');
-	    } catch (Exception $e) {
-		common_debug("damn that sucks");
-		$this->showForm($e->getMessage());
-		return;
-	    }
-
-	    $filename = Avatar::filename($app->id,
-					 image_type_to_extension($imagefile->type),
-					 null,
-					 'oauth-app-icon-'.common_timestamp());
-
-	    $filepath = Avatar::path($filename);
-
-	    move_uploaded_file($imagefile->filepath, $filepath);
-
-	    $app->setOriginal($filename);
-	}
+        $app = Oauth_application::staticGet('name', $name);
+        return !empty($app);
     }
 
 }