From f97c6a56ecdebff2c61e0db8dd58e7760bc2748e Mon Sep 17 00:00:00 2001 From: Clement Desmidt Date: Tue, 17 Mar 2020 17:33:04 +0100 Subject: [PATCH] =?UTF-8?q?:sparkles:=20Permet=20la=20mise=20=C3=A0=20jour?= =?UTF-8?q?=20du=20profil?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit La mise à jour de l'avatar fonctionne à moitié Pour #12 --- app/Http/Controllers/ImageController.php | 4 - app/Http/Controllers/UserController.php | 41 ++ app/Http/Requests/UpdateUser.php | 35 ++ app/User.php | 42 +- composer.json | 3 +- composer.lock | 454 +++++++++++++++++- .../2020_03_17_170058_create_media_table.php | 38 ++ resources/views/layouts/connected.blade.php | 13 +- resources/views/user/index.blade.php | 106 ++++ routes/web.php | 31 +- 10 files changed, 745 insertions(+), 22 deletions(-) create mode 100644 app/Http/Controllers/UserController.php create mode 100644 app/Http/Requests/UpdateUser.php create mode 100644 database/migrations/2020_03_17_170058_create_media_table.php create mode 100644 resources/views/user/index.blade.php diff --git a/app/Http/Controllers/ImageController.php b/app/Http/Controllers/ImageController.php index 44ad80b..6db93c4 100644 --- a/app/Http/Controllers/ImageController.php +++ b/app/Http/Controllers/ImageController.php @@ -23,10 +23,6 @@ class ImageController extends Controller */ public function display($post_id, $options = 'o:full', $image, ImageService $image_service) { - if (Auth::guest()) { - throw new UnauthorizedHttpException('Vous devez être connecté pour voir cette image.'); - } - $post = Post::find($post_id); if (Auth::user()->getAuthIdentifier() !== (int)$post->user_id) { diff --git a/app/Http/Controllers/UserController.php b/app/Http/Controllers/UserController.php new file mode 100644 index 0000000..3733780 --- /dev/null +++ b/app/Http/Controllers/UserController.php @@ -0,0 +1,41 @@ + Auth::user()]); + } + + public function update(UpdateUser $request) + { + /** @var User $user */ + $user_id = Auth::user()->getAuthIdentifier(); + $user = User::find($user_id); + $validated = $request->validated(); + if (!array_key_exists('encrypt_messages', $validated)) { + $validated['encrypt_messages'] = 0; + } + if (isset($validated['avatar'])) { + try { + $user->clearMediaCollection('avatars'); + $user->addMediaFromRequest('avatar')->toMediaCollection('avatars'); + } catch (DiskDoesNotExist $e) { + } catch (FileDoesNotExist $e) { + } catch (FileIsTooBig $e) { + } + } + $user->update($validated); + + return redirect(route('user.index'))->withSuccess('Data saved!'); + } +} diff --git a/app/Http/Requests/UpdateUser.php b/app/Http/Requests/UpdateUser.php new file mode 100644 index 0000000..90493cd --- /dev/null +++ b/app/Http/Requests/UpdateUser.php @@ -0,0 +1,35 @@ + 'required|min:4|max:255', + 'email' => 'required|email:rfc|unique:users,email,'.Auth::user()->getAuthIdentifier(), + 'encrypt_messages' => 'boolean', + 'notification_hour' => 'in:0,1,2,3,4', + 'avatar' => 'file|dimensions:min_width=50,min_height=50,max_width=200,max_height=200,ratio=1' + ]; + } +} diff --git a/app/User.php b/app/User.php index 60bc353..016b6ac 100644 --- a/app/User.php +++ b/app/User.php @@ -5,10 +5,14 @@ namespace App; use Illuminate\Contracts\Auth\MustVerifyEmail; use Illuminate\Foundation\Auth\User as Authenticatable; use Illuminate\Notifications\Notifiable; +use Spatie\Image\Exceptions\InvalidManipulation; +use Spatie\MediaLibrary\HasMedia\HasMedia; +use Spatie\MediaLibrary\HasMedia\HasMediaTrait; +use Spatie\MediaLibrary\Models\Media; -class User extends Authenticatable +class User extends Authenticatable implements HasMedia { - use Notifiable; + use Notifiable, HasMediaTrait; /** * The attributes that are mass assignable. @@ -16,7 +20,7 @@ class User extends Authenticatable * @var array */ protected $fillable = [ - 'name', 'email', 'password', + 'name', 'email', 'password', 'notification_hour', 'encrypt_messages', 'avatar', ]; /** @@ -28,14 +32,18 @@ class User extends Authenticatable 'password', 'remember_token', ]; + protected $dates = [ + 'created_at', 'updated_at', 'email_verified_at', + ]; + /** * The attributes that should be cast to native types. * * @var array */ - protected $casts = [ - 'email_verified_at' => 'datetime', - ]; +// protected $casts = [ +// 'email_verified_at' => 'datetime', +// ]; public function getFolder() { @@ -50,4 +58,26 @@ class User extends Authenticatable { return (bool) $this->encrypt_messages; } + + /** + * @param Media|null $media + */ + public function registerMediaConversions(Media $media = null) + { + try { + $this->addMediaConversion('thumb') + ->width(50) + ->height(50); + } catch (InvalidManipulation $e) { + } + } + + /** + * @param string $alias + * @return string + */ + public function getAvatar($alias = 'thumb') + { + return $this->getFirstMediaUrl('avatars'); + } } diff --git a/composer.json b/composer.json index 4de1904..0f5af1b 100644 --- a/composer.json +++ b/composer.json @@ -13,7 +13,8 @@ "intervention/image": "^2.5", "laravel/framework": "^6.0", "laravel/tinker": "^1.0", - "lavary/laravel-menu": "^1.7" + "lavary/laravel-menu": "^1.7", + "spatie/laravel-medialibrary": "^7.0.0" }, "require-dev": { "facade/ignition": "^1.4", diff --git a/composer.lock b/composer.lock index a25d7a0..f5496db 100644 --- a/composer.lock +++ b/composer.lock @@ -4,7 +4,7 @@ "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies", "This file is @generated automatically" ], - "content-hash": "a8999b4d9f518e7a9608ff59c6ba40e9", + "content-hash": "e3145fece6314548c650ce97ed16de32", "packages": [ { "name": "dnoegel/php-xdg-base-dir", @@ -953,6 +953,128 @@ ], "time": "2019-08-24T11:17:19+00:00" }, + { + "name": "league/glide", + "version": "1.5.0", + "source": { + "type": "git", + "url": "https://github.com/thephpleague/glide.git", + "reference": "a5477e9e822ed57b39861a17092b92553634932d" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/thephpleague/glide/zipball/a5477e9e822ed57b39861a17092b92553634932d", + "reference": "a5477e9e822ed57b39861a17092b92553634932d", + "shasum": "" + }, + "require": { + "intervention/image": "^2.4", + "league/flysystem": "^1.0", + "php": "^5.5 | ^7.0", + "psr/http-message": "^1.0" + }, + "require-dev": { + "mockery/mockery": "~0.9", + "phpunit/php-token-stream": "^1.4", + "phpunit/phpunit": "~4.4" + }, + "type": "library", + "extra": { + "branch-alias": { + "dev-master": "1.1-dev" + } + }, + "autoload": { + "psr-4": { + "League\\Glide\\": "src/" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Jonathan Reinink", + "email": "jonathan@reinink.ca", + "homepage": "http://reinink.ca" + } + ], + "description": "Wonderfully easy on-demand image manipulation library with an HTTP based API.", + "homepage": "http://glide.thephpleague.com", + "keywords": [ + "ImageMagick", + "editing", + "gd", + "image", + "imagick", + "league", + "manipulation", + "processing" + ], + "time": "2019-04-03T23:46:42+00:00" + }, + { + "name": "maennchen/zipstream-php", + "version": "1.2.0", + "source": { + "type": "git", + "url": "https://github.com/maennchen/ZipStream-PHP.git", + "reference": "6373eefe0b3274d7b702d81f2c99aa977ff97dc2" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/maennchen/ZipStream-PHP/zipball/6373eefe0b3274d7b702d81f2c99aa977ff97dc2", + "reference": "6373eefe0b3274d7b702d81f2c99aa977ff97dc2", + "shasum": "" + }, + "require": { + "ext-mbstring": "*", + "myclabs/php-enum": "^1.5", + "php": ">= 7.1", + "psr/http-message": "^1.0" + }, + "require-dev": { + "ext-zip": "*", + "guzzlehttp/guzzle": ">= 6.3", + "mikey179/vfsstream": "^1.6", + "phpunit/phpunit": ">= 7.5" + }, + "type": "library", + "autoload": { + "psr-4": { + "ZipStream\\": "src/" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Paul Duncan", + "email": "pabs@pablotron.org" + }, + { + "name": "Jesse Donat", + "email": "donatj@gmail.com" + }, + { + "name": "Jonatan Männchen", + "email": "jonatan@maennchen.ch" + }, + { + "name": "András Kolesár", + "email": "kolesar@kolesar.hu" + } + ], + "description": "ZipStream is a library for dynamically streaming dynamic zip files from PHP without writing to the disk at all on the server.", + "keywords": [ + "stream", + "zip" + ], + "time": "2019-07-17T11:01:58+00:00" + }, { "name": "monolog/monolog", "version": "2.0.0", @@ -1034,6 +1156,52 @@ ], "time": "2019-08-30T09:56:44+00:00" }, + { + "name": "myclabs/php-enum", + "version": "1.7.6", + "source": { + "type": "git", + "url": "https://github.com/myclabs/php-enum.git", + "reference": "5f36467c7a87e20fbdc51e524fd8f9d1de80187c" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/myclabs/php-enum/zipball/5f36467c7a87e20fbdc51e524fd8f9d1de80187c", + "reference": "5f36467c7a87e20fbdc51e524fd8f9d1de80187c", + "shasum": "" + }, + "require": { + "ext-json": "*", + "php": ">=7.1" + }, + "require-dev": { + "phpunit/phpunit": "^7", + "squizlabs/php_codesniffer": "1.*", + "vimeo/psalm": "^3.8" + }, + "type": "library", + "autoload": { + "psr-4": { + "MyCLabs\\Enum\\": "src/" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "PHP Enum contributors", + "homepage": "https://github.com/myclabs/php-enum/graphs/contributors" + } + ], + "description": "PHP Enum implementation", + "homepage": "http://github.com/myclabs/php-enum", + "keywords": [ + "enum" + ], + "time": "2020-02-14T08:15:52+00:00" + }, { "name": "nesbot/carbon", "version": "2.24.0", @@ -1698,6 +1866,290 @@ ], "time": "2018-07-19T23:38:55+00:00" }, + { + "name": "spatie/image", + "version": "1.7.6", + "source": { + "type": "git", + "url": "https://github.com/spatie/image.git", + "reference": "74535b5fd67ace75840c00c408666660843e755e" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/spatie/image/zipball/74535b5fd67ace75840c00c408666660843e755e", + "reference": "74535b5fd67ace75840c00c408666660843e755e", + "shasum": "" + }, + "require": { + "ext-exif": "*", + "ext-mbstring": "*", + "league/glide": "^1.4", + "php": "^7.0", + "spatie/image-optimizer": "^1.0", + "spatie/temporary-directory": "^1.0.0", + "symfony/process": "^3.0|^4.0|^5.0" + }, + "require-dev": { + "larapack/dd": "^1.1", + "phpunit/phpunit": "^6.0|^7.0", + "symfony/var-dumper": "^3.2|^5.0" + }, + "type": "library", + "autoload": { + "psr-4": { + "Spatie\\Image\\": "src" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Freek Van der Herten", + "email": "freek@spatie.be", + "homepage": "https://spatie.be", + "role": "Developer" + } + ], + "description": "Manipulate images with an expressive API", + "homepage": "https://github.com/spatie/image", + "keywords": [ + "image", + "spatie" + ], + "time": "2020-01-26T18:56:44+00:00" + }, + { + "name": "spatie/image-optimizer", + "version": "1.2.1", + "source": { + "type": "git", + "url": "https://github.com/spatie/image-optimizer.git", + "reference": "9c1d470e34b28b715d25edb539dd6c899461527c" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/spatie/image-optimizer/zipball/9c1d470e34b28b715d25edb539dd6c899461527c", + "reference": "9c1d470e34b28b715d25edb539dd6c899461527c", + "shasum": "" + }, + "require": { + "ext-fileinfo": "*", + "php": "^7.2", + "psr/log": "^1.0", + "symfony/process": "^4.2|^5.0" + }, + "require-dev": { + "phpunit/phpunit": "^8.0", + "symfony/var-dumper": "^4.2|^5.0" + }, + "type": "library", + "autoload": { + "psr-4": { + "Spatie\\ImageOptimizer\\": "src" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Freek Van der Herten", + "email": "freek@spatie.be", + "homepage": "https://spatie.be", + "role": "Developer" + } + ], + "description": "Easily optimize images using PHP", + "homepage": "https://github.com/spatie/image-optimizer", + "keywords": [ + "image-optimizer", + "spatie" + ], + "time": "2019-11-25T12:29:24+00:00" + }, + { + "name": "spatie/laravel-medialibrary", + "version": "7.18.0", + "source": { + "type": "git", + "url": "https://github.com/spatie/laravel-medialibrary.git", + "reference": "6461708267ca8863b351eb70f6d5eb324a3eec0b" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/spatie/laravel-medialibrary/zipball/6461708267ca8863b351eb70f6d5eb324a3eec0b", + "reference": "6461708267ca8863b351eb70f6d5eb324a3eec0b", + "shasum": "" + }, + "require": { + "ext-fileinfo": "*", + "illuminate/bus": "~5.5.0|~5.6.0|~5.7.0|~5.8.0|^6.0", + "illuminate/console": "~5.5.0|~5.6.0|~5.7.0|~5.8.0|^6.0", + "illuminate/database": "~5.5.0|~5.6.0|~5.7.0|~5.8.0|^6.0", + "illuminate/pipeline": "~5.5.0|~5.6.0|~5.7.0|~5.8.0|^6.0", + "illuminate/support": "~5.5.0|~5.6.0|~5.7.0|~5.8.0|^6.0", + "league/flysystem": "^1.0.8", + "maennchen/zipstream-php": "^1.0", + "php": "^7.2", + "spatie/image": "^1.4.0", + "spatie/pdf-to-image": "^1.2", + "spatie/temporary-directory": "^1.1" + }, + "conflict": { + "php-ffmpeg/php-ffmpeg": "<0.6.1" + }, + "require-dev": { + "doctrine/dbal": "^2.5.2", + "ext-pdo_sqlite": "*", + "guzzlehttp/guzzle": "^6.3", + "league/flysystem-aws-s3-v3": "^1.0.13", + "mockery/mockery": "^1.0.0", + "orchestra/testbench": "~3.8.0|^4.0", + "phpunit/phpunit": "^8.0", + "spatie/phpunit-snapshot-assertions": "^2.0" + }, + "suggest": { + "league/flysystem-aws-s3-v3": "Required to use AWS S3 file storage", + "php-ffmpeg/php-ffmpeg": "Required for generating video thumbnails" + }, + "type": "library", + "extra": { + "laravel": { + "providers": [ + "Spatie\\MediaLibrary\\MediaLibraryServiceProvider" + ] + } + }, + "autoload": { + "psr-4": { + "Spatie\\MediaLibrary\\": "src" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Freek Van der Herten", + "email": "freek@spatie.be", + "homepage": "https://murze.be", + "role": "Developer" + } + ], + "description": "Associate files with Eloquent models", + "homepage": "https://github.com/spatie/laravel-medialibrary", + "keywords": [ + "cms", + "conversion", + "downloads", + "images", + "laravel", + "laravel-medialibrary", + "media", + "spatie" + ], + "time": "2020-01-05T12:27:11+00:00" + }, + { + "name": "spatie/pdf-to-image", + "version": "1.8.2", + "source": { + "type": "git", + "url": "https://github.com/spatie/pdf-to-image.git", + "reference": "22a580e03550c95ce810ad430e3c44d2f7a1c75a" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/spatie/pdf-to-image/zipball/22a580e03550c95ce810ad430e3c44d2f7a1c75a", + "reference": "22a580e03550c95ce810ad430e3c44d2f7a1c75a", + "shasum": "" + }, + "require": { + "ext-imagick": "*", + "php": "^7.0" + }, + "require-dev": { + "phpunit/phpunit": "^6.2" + }, + "type": "library", + "autoload": { + "psr-4": { + "Spatie\\PdfToImage\\": "src" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Freek Van der Herten", + "email": "freek@spatie.be", + "homepage": "https://spatie.be", + "role": "Developer" + } + ], + "description": "Convert a pdf to an image", + "homepage": "https://github.com/spatie/pdf-to-image", + "keywords": [ + "convert", + "image", + "pdf", + "pdf-to-image", + "spatie" + ], + "time": "2019-07-31T06:46:19+00:00" + }, + { + "name": "spatie/temporary-directory", + "version": "1.2.2", + "source": { + "type": "git", + "url": "https://github.com/spatie/temporary-directory.git", + "reference": "fcb127e615700751dac2aefee0ea2808ff3f5bb1" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/spatie/temporary-directory/zipball/fcb127e615700751dac2aefee0ea2808ff3f5bb1", + "reference": "fcb127e615700751dac2aefee0ea2808ff3f5bb1", + "shasum": "" + }, + "require": { + "php": "^7.2" + }, + "require-dev": { + "phpunit/phpunit": "^8.0" + }, + "type": "library", + "autoload": { + "psr-4": { + "Spatie\\TemporaryDirectory\\": "src" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Alex Vanderbist", + "email": "alex@spatie.be", + "homepage": "https://spatie.be", + "role": "Developer" + } + ], + "description": "Easily create, use and destroy temporary directories", + "homepage": "https://github.com/spatie/temporary-directory", + "keywords": [ + "spatie", + "temporary-directory" + ], + "time": "2019-12-15T18:52:09+00:00" + }, { "name": "swiftmailer/swiftmailer", "version": "v6.2.1", diff --git a/database/migrations/2020_03_17_170058_create_media_table.php b/database/migrations/2020_03_17_170058_create_media_table.php new file mode 100644 index 0000000..139ec80 --- /dev/null +++ b/database/migrations/2020_03_17_170058_create_media_table.php @@ -0,0 +1,38 @@ +bigIncrements('id'); + $table->morphs('model'); + $table->string('collection_name'); + $table->string('name'); + $table->string('file_name'); + $table->string('mime_type')->nullable(); + $table->string('disk'); + $table->unsignedBigInteger('size'); + $table->json('manipulations'); + $table->json('custom_properties'); + $table->json('responsive_images'); + $table->unsignedInteger('order_column')->nullable(); + $table->nullableTimestamps(); + }); + } + + /** + * Reverse the migrations. + */ + public function down() + { + Schema::dropIfExists('media'); + } +} diff --git a/resources/views/layouts/connected.blade.php b/resources/views/layouts/connected.blade.php index ee9b641..501171e 100644 --- a/resources/views/layouts/connected.blade.php +++ b/resources/views/layouts/connected.blade.php @@ -55,15 +55,22 @@
- {{ substr(Auth::user()->name, 0, 1) }} + @php($avatar = Auth::user()->getAvatar()) + + @if($avatar !== '') + + @else + {{ substr(Auth::user()->name, 0, 1) }} + @endif + {{ Auth::user()->name }}
- - Profile + + {{ __('Profile') }} Settings diff --git a/resources/views/user/index.blade.php b/resources/views/user/index.blade.php new file mode 100644 index 0000000..3f924dc --- /dev/null +++ b/resources/views/user/index.blade.php @@ -0,0 +1,106 @@ +@extends('layouts.connected') + +@section('content') +
+
+
+
+

+ {{ __('My profile') }} +

+ @if (session('status')) + + @endif + @if (session('success')) + + @endif + @if (session('errors')) + + @endif +
+
+
+
+
+
+ @csrf +
+
+
+
+ + +
+
+ + +
+
+ +
*********
+
+
+ +
+
+ +
+ + + + + +
+
+
+
+
+
{{ __('Avatar') }}
+ @php($avatar = Auth::user()->getAvatar()) + @if($avatar !== '') + + @endif +
+ + +
+
+
+
+
+
+
+ {{ __('Cancel') }} + +
+
+
+
+
+
+@endsection diff --git a/routes/web.php b/routes/web.php index 17626f2..c996b05 100644 --- a/routes/web.php +++ b/routes/web.php @@ -11,13 +11,30 @@ | */ +use Illuminate\Support\Facades\Auth; +use Illuminate\Support\Facades\Route; + Auth::routes(); +// not connected Route::get('/', 'WelcomeController@index')->name('home'); -Route::get('/home', 'DashboardController@index')->name('dashboard'); -Route::post('store', 'DashboardController@store')->name('store'); -Route::post('delete', 'DashboardController@delete')->name('delete'); -Route::post('edit', 'DashboardController@edit')->name('edit'); -Route::post('stats', 'StatsController@tag')->name('stats'); // TODO make a group for stats -Route::get('/display/{post_id}/{options}/{image_name}', 'ImageController@display')->name('display_image'); -Route::get('/gallery', 'ImageController@gallery')->name('gallery'); + +// connected +Route::group(['middleware' => ['auth']], function() { + // posts + Route::get('/home', 'DashboardController@index')->name('dashboard'); + Route::post('store', 'DashboardController@store')->name('store'); + Route::post('delete', 'DashboardController@delete')->name('delete'); + Route::post('edit', 'DashboardController@edit')->name('edit'); + + // stats + Route::post('stats', 'StatsController@tag')->name('stats'); // TODO make a group for stats + + // gallery + Route::get('/display/{post_id}/{options}/{image_name}', 'ImageController@display')->name('display_image'); + Route::get('/gallery', 'ImageController@gallery')->name('gallery'); + + // user + Route::get('/user', 'UserController@index')->name('user.index'); + Route::post('/user', 'UserController@update')->name('user.update'); +});