Attaching Media form spate media library #42

Activity

1. toonvandenbos commented on Aug 23, 2019

   

   toonvandenbos

   on Aug 23, 2019

   Member

   More actions

   Hi @OliverZiegler,

   The problem here is that Spatie's library requires a model in order to link the media to it. By default, this package does not use models (data is stored in JSON files). Therefore, you should switch to database storage. This will however probably not work with the current version of nova-flexible-content, since the model is difficult to fetch from inside a Flexible content.

   I'm working on a fix for a similar issue (the one you linked in your post) as we speak, I'll try to find a solution that also fixes this one.

   👍React with 👍1Reacted by Thomas Swonke
2. toonvandenbos

   added a commit that references this issue on Aug 23, 2019

   Added Model for database source. Added Template::getOriginal method f…

   ...

   7d0d15d
3. toonvandenbos commented on Aug 23, 2019

   

   toonvandenbos

   on Aug 23, 2019

   Last edited by toonvandenbos

   Member

   More actions

   Hi @OliverZiegler,

   Could you update both packages (using dev-master for nova-page) and tell me if this issue is fixed?

   Here's how I got it working (you should probably re-publish the package's configuration file):

   1. Switch Nova-Page's "default_source" configuration to Database ;
   2. Create a new model like this:

   namespace App;
   
   use Whitecube\NovaPage\Sources\StaticPage;
   use Spatie\MediaLibrary\HasMedia\HasMedia;
   use Spatie\MediaLibrary\HasMedia\HasMediaTrait;
   
   class Page extends StaticPage implements HasMedia
   {
       use HasMediaTrait;
   }

   3. Go back to nova-page's configuration and set sources.database.model to your new Page model (or whatever you named it).
4. OliverZiegler commented on Aug 26, 2019

   

   OliverZiegler

   on Aug 26, 2019

   Author

   More actions

   Thanks for the fast response!!

   So with the still missing PR from advanced-nova-media-library I got it working with nova-flexible-content but with some limitations...

   • Adding the same Layout multiple times overwrites the images (related to this issue I think.)
   • Defining single file collections won't work (also related to this issue

   One more issue (not related to nova-flexible-content) is adding Images to the page directly.
   Defining advanced-nova-medialibrary fields on the Template worked for me (inspired by your trait in nova-flexible-content) by adding the following code to the template:

   Make the template implement use Spatie\MediaLibrary\HasMedia\HasMedia; and adding the trait with use Spatie\MediaLibrary\HasMedia\HasMediaTrait;.

   Furthermore adding:

       protected function getMediaModel() : HasMedia
       {
           $model = $this->getSource()->getOriginal($this);
   
           \Log::info($model);
   
           if(is_null($model) || !($model instanceof HasMedia)) {
               throw new \Exception('Origin HasMedia model not found.');
           }
   
           return $model;
       }
   
   
       public function getMedia(string $collectionName = 'default', $filters = [])
       {
           return app(MediaRepository::class)->getCollection($this->getMediaModel(), $collectionName, $filters);
       }
   
       public function addMedia($file)
       {
           return app(FileAdderFactory::class)
               ->create($this->getMediaModel(), $file)
               ->preservingOriginal();
       }

   ❤️React with ❤️1Reacted by Raphael ASTIER
5. GarethSomers commented on Sep 3, 2019

   

   GarethSomers

   on Sep 3, 2019

   Last edited by GarethSomers

   Contributor

   More actions

   Hey guys,

   When running:

   • whitecube/nova-page:dev-master
   • https://github.com/whitecube/nova-flexible-content/pull/71 whitecube/nova-flexible-content@cfd8719

   Pressing save won't do anything 😕

   Update

   My fault, ignore me :)

   😄React with 😄1Reacted by Toon Van den Bos
6. amidesfahani commented on Oct 11, 2020

   

   amidesfahani

   on Oct 11, 2020

   More actions

   does it world?

   😄React with 😄2Reacted by Artūras and Hassan Nawaz