Browse Source

Merge pull request #3 from electerious/develop

Develop
Quentin Ligier 8 years ago
parent
commit
e39d84f1b2
5 changed files with 7 additions and 8 deletions
  1. 0 0
      dist/main.js
  2. 1 1
      php/Modules/Album.php
  3. 1 1
      php/Modules/Photo.php
  4. 5 0
      php/helpers/getExtension.php
  5. 0 6
      src/scripts/upload.js

File diff suppressed because it is too large
+ 0 - 0
dist/main.js


+ 1 - 1
php/Modules/Album.php

@@ -278,7 +278,7 @@ final class Album {
 			if (!@is_readable($photo->url)) continue;
 
 			// Get extension of image
-			$extension = getExtension($photo->url, true);
+			$extension = getExtension($photo->url, false);
 
 			// Set title for photo
 			$zipFileName = $zipTitle . '/' . $photo->title . $extension;

+ 1 - 1
php/Modules/Photo.php

@@ -845,7 +845,7 @@ final class Photo {
 		}
 
 		// Get extension
-		$extension = getExtension($photo->url, true);
+		$extension = getExtension($photo->url, false);
 		if (empty($extension)===true) {
 			Log::error(Database::get(), __METHOD__, __LINE__, 'Invalid photo extension');
 			return false;

+ 5 - 0
php/helpers/getExtension.php

@@ -11,6 +11,11 @@ function getExtension($filename, $isURI = false) {
 
 	$extension = pathinfo($filename, PATHINFO_EXTENSION);
 
+	// Special cases
+	// https://github.com/electerious/Lychee/issues/482
+	list($extension) = explode(':', $extension, 2);
+	list($extension) = explode('?', $extension, 2);
+
 	if (empty($extension)===false) $extension = '.' . $extension;
 
 	return $extension;

+ 0 - 6
src/scripts/upload.js

@@ -275,12 +275,6 @@ upload.start = {
 
 				basicModal.close()
 
-				let extension = data.link.split('.').pop()
-				if (extension!=='jpeg' && extension!=='jpg' && extension!=='png' && extension!=='gif' && extension!=='webp') {
-					loadingBar.show('error', 'File format of link not supported.')
-					return false
-				}
-
 				files[0] = {
 					name      : data.link,
 					supported : true

Some files were not shown because too many files changed in this diff