[php+symfony] Error: Call to a member function move() on a non-object Opcje

[dopelganger]

cześć
dodałem do profilu użytkownika możliwość upload obrazka (avatar), no i pięknie działa. Ale po zalogowaniu się do konta a następnie przekierowaniu -> wywala bład:

Error: Call to a member function move() on a non-object

czepia się o linię którą wyszczałkowałem Prosze o pomoc.


Kod klasy User.php

[PHP] pobierz, plaintext 
 
	public function getFullImagePath() {
        return null === $this->image ? null : $this->getUploadRootDir(). $this->image;
    }
 
    protected function getUploadRootDir() {
        // the absolute directory path where uploaded documents should be saved
        return $this->getTmpUploadRootDir().$this->getId()."/";
    }
 
    protected function getTmpUploadRootDir() {
        // the absolute directory path where uploaded documents should be saved
        return __DIR__ . '/../../../../web/images/uploads/';
    }
 
    /**
     * @ORM\PrePersist()
     * @ORM\PreUpdate()
     */
    public function uploadImage() {
        // the file property can be empty if the field is not required
        if (null === $this->image) {
            return;
        }
        if(!$this->id){
            $this->image->move($this->getTmpUploadRootDir(), $this->image->getClientOriginalName());
        }else{
           $this->image->move($this->getUploadRootDir(), $this->image->getClientOriginalName()); // <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
        }
        $this->setImage($this->image->getClientOriginalName());
    }
 
    /**
     * @ORM\PostPersist()
     */
    public function moveImage()
    {
        if (null === $this->image) {
            return;
        }
        if(!is_dir($this->getUploadRootDir())){
            mkdir($this->getUploadRootDir());
        }
        copy($this->getTmpUploadRootDir().$this->image, $this->getFullImagePath());
        unlink($this->getTmpUploadRootDir().$this->image);
    }
 
    /**
     * @ORM\PreRemove()
     */
    public function removeImage()
    {
        unlink($this->getFullImagePath());
        rmdir($this->getUploadRootDir());
    }
[PHP] pobierz, plaintext 

Ten post edytował dopelganger 23.09.2015, 13:46:50

[viking]

Wstaw

[PHP] pobierz, plaintext 
var_dump($this->image);
[PHP] pobierz, plaintext 

linię wcześniej.

[thek]

Ja bym już $this sprawdzał

[dopelganger]

Wstaw

[PHP] pobierz, plaintext 
var_dump($this->image);
[PHP] pobierz, plaintext 

linię wcześniej.

pojawia mi się:

string(8) "test.jpg"

[memory]

[PHP] pobierz, plaintext 
 $this->setImage($this->image->getClientOriginalName()); 
[PHP] pobierz, plaintext 

ustawiasz zmienną która powinna być UploadedFile stringiem. Usuń to

Powinno być wyżej

[PHP] pobierz, plaintext 
/**
     * Sets file.
     *
     * @param UploadedFile $file
     */
    public function setImage(UploadedFile $image= null)
    {
        $this->image= $image;
    }
[PHP] pobierz, plaintext 

Dokładnie przeczytaj

http://symfony.com/doc/current/cookbook/do...le_uploads.html

Ten post edytował memory 24.09.2015, 07:49:09

[dopelganger]

[PHP] pobierz, plaintext 
 $this->setImage($this->image->getClientOriginalName()); 
[PHP] pobierz, plaintext 

ustawiasz zmienną która powinna być UploadedFile stringiem. Usuń to

Powinno być wyżej

[PHP] pobierz, plaintext 
/**
     * Sets file.
     *
     * @param UploadedFile $file
     */
    public function setImage(UploadedFile $image= null)
    {
        $this->image= $image;
    }
[PHP] pobierz, plaintext 

Dokładnie przeczytaj

http://symfony.com/doc/current/cookbook/do...le_uploads.html

nadal ten sam byk

[memory]

Podaj całą zawartość klasy

[dopelganger]

Podaj całą zawartość klasy

[PHP] pobierz, plaintext 
<?php
 
namespace Test\UserBundle\Entity;
 
use FOS\UserBundle\Entity\User as BaseUser;
use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Validator\Constraints as Assert;
 
 
/**
 * @ORM\Entity
 * @ORM\HasLifecycleCallbacks
 * @ORM\Table(name="Test_users")
 */
 
class User extends BaseUser
{
    /**
     * @ORM\Id
     * @ORM\Column(type="integer")
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    protected $id;
 
      /**
     * @var string $firstname
     *
     * @ORM\Column(name="firstname",type="string",length=30)
     */
    protected $firstname;
 
 
 
	/**
     * @var string $image
     * @Assert\File( maxSize = "15k", mimeTypes = {"image/jpeg", "image/jpg", "image/gif", "image/png", "image/bmp"}, mimeTypesMessage = "Prosze wybrać plik graficzny!")
	 * @ORM\Column(name="image", type="string", length=255)
     */
    protected $image;
 
 
 
 
 
    /**
     * Set firstname
     *
     * @param string $firstname
     * @return User
     */
    public function setFirstname($firstname)
    {
        $this->firstname = $firstname;
 
        return $this;
    }
 
 
    /**
     * Sets image.
     *
     * @param UploadedFile $image
     */
    public function setImage(UploadedFile $image = null)
    {
        $this->image = $image;
    }
 
 
 
 
 
 
 
	/**
     * Get id
     *
     * @return integer
     */
    public function getId()
    {
        return $this->id;
    }
 
 
 
    /**
     * Get firstname
     *
     * @return string
     */
    public function getFirstname()
    {
        return $this->firstname;
    }
 
 
 
    /**
     * Get image.
     *
     * @return UploadedFile
     */
    public function getImage()
    {
        return $this->image;
    }
 
 
 
 
    public function __construct()
    {
        parent::__construct();
        // your own logic
    }
 
 
 
 
 
	// Callbacks - upload ***********************************************************************
 
 
 
	public function getFullImagePath() {
        return null === $this->image ? null : $this->getUploadRootDir(). $this->image;
    }
 
    protected function getUploadRootDir() {
        // the absolute directory path where uploaded documents should be saved
        return $this->getTmpUploadRootDir().$this->getId()."/";
    }
 
    protected function getTmpUploadRootDir() {
        // the absolute directory path where uploaded documents should be saved
        return __DIR__ . '/../../../../web/images/uploads/';
    }
 
    /**
     * @ORM\PrePersist()
     * @ORM\PreUpdate()
     */
    public function uploadImage() {
        // the file property can be empty if the field is not required
        if (null === $this->image) {
            return;
        }
        if(!$this->id){
            $this->image->move($this->getTmpUploadRootDir(), $this->image->getClientOriginalName());
        }else{
			var_dump($this->image);			
            $this->image->move($this->getUploadRootDir(), $this->image->getClientOriginalName());
        }
        $this->setImage($this->image->getClientOriginalName());
    }
 
    /**
     * @ORM\PostPersist()
     */
    public function moveImage()
    {
        if (null === $this->image) {
            return;
        }
        if(!is_dir($this->getUploadRootDir())){
            mkdir($this->getUploadRootDir());
        }
        copy($this->getTmpUploadRootDir().$this->image, $this->getFullImagePath());
        unlink($this->getTmpUploadRootDir().$this->image);
    }
 
    /**
     * @ORM\PreRemove()
     */
    public function removeImage()
    {
        unlink($this->getFullImagePath());
        rmdir($this->getUploadRootDir());
    }
 
 
 
}
[PHP] pobierz, plaintext 

zaznacze że zmieniając na:

setImage(UploadedFile $image = null)

w ogóle nie działa upload ani dodawanie do bazy nazwy pliku, wcześniejsza wersja działała

[kapslokk]

[PHP] pobierz, plaintext 
	/**
     * @var string $image
     * @Assert\File( maxSize = "15k", mimeTypes = {"image/jpeg", "image/jpg", "image/gif", "image/png", "image/bmp"}, mimeTypesMessage = "Prosze wybrać plik graficzny!")
	 * @ORM\Column(name="image", type="string", length=255)
     */
[PHP] pobierz, plaintext 

Chyba już tutaj masz źle, wydaje mi się że $image powinno mieć tylko @Assert, bez kolumny w bazie itd. Zresztą tak jest nawet w tutorialu od symfony. Do bazy zapisujesz np ścieżkę na dysku w osobnym polu.

Ten post edytował kapslokk 24.09.2015, 08:26:38

[dopelganger]

[PHP] pobierz, plaintext 
	/**
     * @var string $image
     * @Assert\File( maxSize = "15k", mimeTypes = {"image/jpeg", "image/jpg", "image/gif", "image/png", "image/bmp"}, mimeTypesMessage = "Prosze wybrać plik graficzny!")
	 * @ORM\Column(name="image", type="string", length=255)
     */
[PHP] pobierz, plaintext 

Chyba już tutaj masz źle, wydaje mi się że $image powinno mieć tylko @Assert, bez kolumny w bazie itd. Zresztą tak jest nawet w tutorialu od symfony. Do bazy zapisujesz np ścieżkę na dysku w osobnym polu.

tylko że zapisuje tutaj nazwę pliku i to działa, działa również upload, wywala mi ten błąd po logowaniu i redirect na szablon profilu. Logowanie zadziała, ale musze cofnąć się do jakiejś innej strony i już wtedy jest ok.

Ten post edytował dopelganger 24.09.2015, 08:30:34

[kapslokk]

Zapisuje, ale w 'image' przez to masz string, a nie obiekt. Na stringu nie zrobisz ->move, a na obiekcie owszem.

[memory]

Przeczytaj dokładnie dokumentacje. Czemu do zmiennej $this->image przypisujesz dwie operacje ? zapis nazwy zdjecia oraz klase uploadedfile. Musisz to rozdzielić

[PHP] pobierz, plaintext 
    * @Assert\File( maxSize = "15k", mimeTypes = {"image/jpeg", "image/jpg", "image/gif", "image/png", "image/bmp"}, mimeTypesMessage = "Prosze wybrać plik graficzny!")
 
     */
    protected $file;
 
 
 * @ORM\Column(name="image", type="string", length=255)
     */
    protected $image;
 
 public function setImage($image )
    {
        $this->image = $image;
    }
 
 public function setFile(UploadedFile $file= null)
    {
        $this->file= $file;
    }
 
 /**
     * @ORM\PrePersist()
     * @ORM\PreUpdate()
     */
    public function uploadImage() {
        // the file property can be empty if the field is not required
        if (null === $this->file) {
            return;
        }
        if(!$this->id){
            $this->file->move($this->getTmpUploadRootDir(), $this->file->getClientOriginalName());
        }else{
 
            $this->file->move($this->getUploadRootDir(), $this->file->getClientOriginalName());
        }
        $this->setImage($this->file->getClientOriginalName());
    }
 
 public function moveImage()
    {
        if (null === $this->file) {
            return;
        }
        if(!is_dir($this->getUploadRootDir())){
            mkdir($this->getUploadRootDir());
        }
        copy($this->getTmpUploadRootDir().$this->image, $this->getFullImagePath());
        unlink($this->getTmpUploadRootDir().$this->image);
    }
 
 
 
[PHP] pobierz, plaintext 

I zmiana w createFormBuilder image na file

Ten post edytował memory 24.09.2015, 08:39:47

[dopelganger]

wywala mi błąd:

Catchable Fatal Error: Argument 1 passed to Test\UserBundle\Entity\User::setFile() must be an instance of Test\UserBundle\Entity\UploadedFile, instance of Symfony\Component\HttpFoundation\File\UploadedFile given, called in /home/www2/tests.../vendor/symfony/symfony/src/Symfony/Component/PropertyAccess/PropertyAccessor.php on line 410 and defined

przywala się do

[PHP] pobierz, plaintext 
public function setFile(UploadedFile $file= null) {
[PHP] pobierz, plaintext 

w CreateBuiler mam:

[PHP] pobierz, plaintext 
$builder->add('file', null, array('data_class' => null, 'label' => 'form.file','translation_domain' => 'FOSUserBundle', 'required'=>false));
[PHP] pobierz, plaintext 

[memory]

[PHP] pobierz, plaintext 
$builder->add('file', 'file', array('data_class' => 'Test\UserBundle\Entity\User', 'label' => 'form.file','translation_domain' => 'FOSUserBundle', 'required'=>false));
[PHP] pobierz, plaintext 

Spróbuj tak

[dopelganger]

[PHP] pobierz, plaintext 
$builder->add('file', 'file', array('data_class' => 'Test\UserBundle\Entity\User', 'label' => 'form.file','translation_domain' => 'FOSUserBundle', 'required'=>false));
[PHP] pobierz, plaintext 

Spróbuj tak

nadal wyskakuje błąd

[ohm]

Kod

use Symfony\Component\HttpFoundation\File\UploadedFile;

?

[dopelganger]

Kod

use Symfony\Component\HttpFoundation\File\UploadedFile;

?

po tym zabiegu błąd się nie pojawia, ale nie zapisuje nazwy pliku do bazy, ani nie wgrywa pliku na serwer

[kpt_lucek]

Eh

Rozdziel to na 2 przypadki:
1. masz model który jest klasą DTO i taki powinien zostać, czyste get,set,add,remove, żadnych operacji na plikach.
2. masz usługę która na podstawie w/w obiektu ogarnia save/remove/move itd. obrazka

Dzięki temu oddzielisz logikę od modelu i obie drogi będziesz mógł rozwijać niezależnie, bo po co sobie utrudniać?

P.S.

Zaintersuj się bibliotekami do zarządzania obrazkami, jest ich masa na GH i/lub Packagist

Ten post edytował kpt_lucek 24.09.2015, 14:05:02

[dopelganger]

cześć
sory, ale wracam jeszcze do tego tematu.
Przetestowałem 3 różne sposoby tego uploadu łącznie z tym który tutaj wrzuciłem a potem poprawiłem wg waszych wskazówek i znalazłem pewną (błędną?) prawidłowość w każdym ze sposobów:

a więc:
edytuje profil usera, wskazuje obrazek z dysku, zapisuje dane i JEST OK, => ALE KIEDY ponownie to robie dla danego usera już nie zapisuje obrazka do bazy ani go nie uploaduje, nawet kiedy usune nazwę pliku z bazy :/
I ZNOWU kiedy tworze nowy / kolejny profil a potem aktualizuje jego dane - plik wczytuje na serwer i zapisuje do bazy - ale udaje się tylko za pierwszym razem.. itd itd itd
taka pętla...

to dotyczy jakiegokolwiek przykładu wg którego się wzoruję lub testuje - łącznie z manualem http://symfony.com/doc/current/cookbook/do...le_uploads.html


dziwne zjawisko

może ktoś coś wie ?

Ten post edytował dopelganger 8.10.2015, 12:39:12