Using the new OneToMany Collections

Everything in this lesson works great for me right up until the end (time 5:27 in your video) where I click the edit link on the admin/genus page. Things blow up with "Cannot access private property AppBundle\Entity\GenusScientist::$id". I can't figure out why this is happening. I do have a "public function getId()" in my GenusScientist entity. I even copied your final version of this entity from the course files and the same thing happens, so this has to be coming from somewhere else.

Any hints?

For what it's worth, if I change the scope of $id to public (rather than private) in the GenusScientist entity, then all works fine.

Ah, it was good for debugging, now we know exactly what the problem is, but we definitely should find the place where the private `id` property is called outside the GenusScientist class and fix it.

Cheers!

Hi Victor, i just can say about it that the problem is the same for me and i found the problem, but i can t understand it. In the code of GenusFormType.php, if i erase the line

'expanded' => true,

from the

->add('genusScientists', EntityType::class, [
'class' => User::class,
'multiple' => true,
'expanded' => true,
'choice_label' => 'email',
'query_builder' => function(UserRepository $repo){
return $repo->createIsScientistQueryBuilder();
}
])

it works perfect (of course without checkboxes).
Also in the next lesson that you re changing the code there is none problem at all!

I hope it helped a little bit!!! Keep going with the perfect work ;)

Yo argy_13!

Thanks for sharing! Yea... this is a mystery indeed :). And this is an interesting clue! Before you posted this, I was pretty confident that "expanded" is really a superficial option: it just controls how the elements are rendered... and not much more. But, now I think I'm wrong! I guess it's not a huge deal, but if you're able to get this error again, a screenshot of the full stacktrace might be enough to crack it finally!

Cheers!

Hi Ryan!
I tried to change just this part again (because i finished the 21st lesson, so i changed my code for 3 more lesson and i don t know the consequences to this error). I put back the code for

->add('genusScientists',....

with 'expanded' as before and i got the same error, but i don t know if it mention the same problems.. here is the page with the error

Hey argy_13

I think you forgot to paste the page's url :)

Diego, I pasted the whole page but it didn t upload these 2 replies... i saw it right now! Maybe because it was too big..
I will try again, but do you want to send me an email adress to send both the document and the print screen?

Would be easier if you upload it to https://pastebin.com/ and share me the URL :)

https://pastebin.com/92nBGTWs

i hope that this will going to work!!

Hey argy_13!

Sorry for the late response. This is very interesting, for some reason it cannot access to your ID property via reflection, I would like to test something, change the scope to public and try it again.

// GenusScientist.php
...
public $id;
...

Hi Diego, my reply is also late, but sorry! :)
Unfortunately, i m not in this chapter any more, but the next days when i ll find time i will try to put it in the specific point and then turn it to public to give you the errors. I hope i will do it soon that you find the bug.

Cheers!

Hi Diego,
i did all the lessons just to help you with this!

If i do it

// GenusScientist.php
...
public $id;

it will work perfectly.

Also, if i remove the line

'expanded' => true,

from the GenusFormType.php

If you want something else from me i can keep it for 3-4 days more like this!

I m sure that you ll find the solution! ;)

Hey argy_13

Thanks for taking your time to answer me :)

As I expected, it just cannot access to the property because something weird happens when setting to true the 'expanded' option... I don't know why this happens but it's very interesting!

It s ok, Diego! if you want to ask me to do any test and you say me what i have to examine, you can say it! ;-)

Just to mention that i have the exact same error message in the 23rd lesson at the 4:03 when we try to reach the edit for the update user (you don t use it in the lesson because you re erasing all the code for save genus from User s entity)

Cannot access private property AppBundle\Entity\GenusScientist::$id

Also the problem is the 'expanded' from the UserEditForm

If you want also the

Stack Trace (Plain Text)
i have saved it to a document so i can upload it

I hope i helped a litlle bit
Keep your wonderful work ;)

I have the same issue, it has to be something in the GenusAdminController editAction or in the GenusFormType

Hey Daniel,

Yes, it makes sense. Try to find the private `id` property call in those files. If you don't find any calls, then you can try my suggestion here: https://knpuniversity.com/s...

Cheers!

Hey Ernest H. ,

Hm, it's weird... if you sure you have the exactly public function getId() inside of GenusScientist entity, so I bet you have a direct call of ->id somewhere in your PHP files, because Twig should resolve the public getter method automatically when you call {{ genusScientist.id }}. I think the easiest way is to find all ->id calls in your PHP files. If you use PhpStorm, then you can select src/ dir from the project files in the right sidebar and press Shift + Command + F on Mac (or right click on src/ and then choose "Find in path..."). Then enter the ->id text to find and check the result. Let me know if you find some "->id" calls on a GenusScientist object except the one inside the getId() method.

Cheers!

Hi Ryan (or any of his colleagues)
I have spent over a day in trying to resolve this issue and it really start to frustrate me.
Basically I watched the tutorial of "ManyToMany with extra field" a bunches of time and followed the tutorial by the letter, but I can't seems to be able to save/remove items.
A bit of history. I have a User entity and a Social Entity. Originally, I created a new field in the User called "socials" having ManyToMany relationship with the Social Entity. This created all methods needed in the User and Social classes, as well as creating an additional UserSocial entity.
I then followed the tutorial to change the ManyToMany into OneToMany, but when I try to edit a user profile, I cannot seems to get around the saving/removing problem.
Here is what I have:

Entity\User
    /**
     * @ORM\OneToMany(targetEntity="App\Entity\UserSocial", mappedBy="user")
     */
    private $socials;

    /**
     * @return Collection|Social[]
     */
    public function getSocials(): Collection
    {
        return $this->socials;
    }

    public function addSocial(Social $social): self
    {
        if (!$this->socials->contains($social)) {
            $this->socials[] = $social;
        }

        return $this;
    }

    public function removeSocial(Social $social): self
    {
        if ($this->socials->contains($social)) {
            $this->socials->removeElement($social);
        }

        return $this;
    }

Entity\Social

class Social
{

/**
 * @ORM\Id()
 * @ORM\GeneratedValue()
 * @ORM\Column(type="integer")
 */
private $id;

/**
 * @ORM\Column(type="string", length=100)
 */
private $name;

/**
 * @ORM\Column(type="string", length=100, nullable=true)
 */
private $icon;

/**
 * @ORM\OneToMany(targetEntity="App\Entity\UserSocial", mappedBy="social")
 */
private $users;

public function __construct()
{
    $this->users = new ArrayCollection();
}

public function getId(): ?int
{
    return $this->id;
}

public function getName(): ?string
{
    return $this->name;
}

public function setName(string $name): self
{
    $this->name = $name;

    return $this;
}

public function getIcon(): ?string
{
    return $this->icon;
}

public function setIcon(?string $icon): self
{
    $this->icon = $icon;

    return $this;
}

public function __toString()
{
    return (string) $this->getName();
}

/**
 * @return Collection|User[]
 */
public function getUsers(): Collection
{
    return $this->users;
}

public function addUser(User $user): self
{
    if (!$this->users->contains($user)) {
        $this->users[] = $user;
        $user->addSocial($this);
    }

    return $this;
}

public function removeUser(User $user): self
{
    if ($this->users->contains($user)) {
        $this->users->removeElement($user);
        $user->removeSocial($this);
    }

    return $this;
}

}

Entity\UserSocial
/**
 * @ORM\Entity
 * @ORM\Table(name="user_social")
 */
class UserSocial
{
    /**
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     * @ORM\Column(type="integer")
     */
    private $id;

    /**
     * @ORM\ManyToOne(targetEntity="App\Entity\User", inversedBy="socials")
     * @ORM\JoinColumn(nullable=false)
     */
    private $user;

    /**
     * @ORM\ManyToOne(targetEntity="App\Entity\Social", inversedBy="users")
     * @ORM\JoinColumn(nullable=false)
     */
    private $social;
    /**
     * @ORM\Column(type="string")
     */
    private $url;

    /**
     * @return mixed
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * @return mixed
     */
    public function getUser()
    {
        return $this->user;
    }

    /**
     * @param mixed $user
     */
    public function setUser($user): void
    {
        $this->user = $user;
    }

    /**
     * @return mixed
     */
    public function getSocial()
    {
        return $this->social;
    }

    /**
     * @param mixed $social
     */
    public function setSocial($social): void
    {
        $this->social = $social;
    }

    /**
     * @return mixed
     */
    public function getUrl()
    {
        return $this->url;
    }

    /**
     * @param mixed $url
     */
    public function setUrl($url): void
    {
        $this->url = $url;
    }

}

In my ProfileFormType, I have 'socials' mapped as CollectionType

            ->add('socials', CollectionType::class, [
                'entry_type' => TextType::class,
                'allow_add' => true
            ])

in my ProfileController, I have something like that:

$socialMediaList = $em->getRepository('App:Social')->findAllWithUserValues();
$userProfileModel =  new ProfileFormModel();
$userProfileModel->socials = $socialMediaList;
`</pre >

The findAllWithUserValues  has a raw SQL query that retrieve all information I needed (I used the raw SQL because I spent hours in trying to figure out how to use DQL, but with no success

public function findAllWithUserValues()
{

    $conn = $this->getEntityManager()->getConnection();
    $sql = '
        SELECT social.*, us.url 
        FROM social 
        LEFT JOIN user_social us ON social.id = us.social_id AND us.user_id = 1
    ';
    $stmt = $conn->prepare($sql);
    $stmt->execute();
    return $stmt->fetchAll();

}

end in twig, I render it like this:

{% set vars = profileEditForm.socials.vars %}
{% set socialMediaList = profileEditForm.socials.vars.data %}
{% for social in socialMediaList %}
<input type="text" name="{{ vars.full_name }}[{{ social.name }}]" {{ vars.attr.required is defined ? 'required' }} class="form-control" placeholder="{{ social.name }}" value="{{ social.url ?: '' }}" />

Now, what I noticed is that the actual data I need to add is inside the user_social table, therefore, when I do $user->addSocial(), it expect a Social, but I am confused because it would make sense to pass a UserSocial instance. I actually tried to change it so that it expects a UserSocial, but by doing that, I also had to add "cascade={"persist"}. If I did that, it would actually save it to user_social table, but then I got all confused on the removal of the Social from that table.

I am sorry if this thread is very long, but I really hope you can help. I have tried help on skack, google, codementor and I can't get my head around the proper way of saving/removing a ManyToMany with extra field.

Thanks a lot (and sorry again for this long message).

Hey Alessandro,

First of all, I'd recommend you to rename some field. If we're talking about a field that contain collection of UserSocial entities, then call the field as $userSocials instead of just $socials, especially when you have Social entity. This will not mislead you what exactly entity we're talking about: UserSocial or Social. It's minor , but just a friendly advice. Also, after all complex manipulations... or actually after any manipulations with entity mapping I'd recommend you to run a special command that helps to validation if your mapping is valid and if your DB is in sync with the current mapping. Run "bin/console doctrine:schema:validate" - if it's green - perfect! Less chances you did something wrong. If it's red - then most probably your problem is related to the output. Try to figure out what's wrong exactly and fix it. So, first of all, please, double check it so we know mapping is correct. At the first sight it looks good for me.

I noticed that you use "model" in controller, like ProfileFormModel. Are you building the form type base on this model? If so, like if you map the form to a model, not entity, then you will need to write a custom logic where you will handle the model and map all the data to real entities, like iterate over collections and create new real entities for them. And of course, every new entity you would need to persist first before calling flush. You may want to debug the final entity, use "dd()" or "dump(); die;" to dump the final data and manually check if all the fields are set correctly.

But if you map your form type to an entity - then try to debug the final data at least to make sure they look good.

So, yes, I see why you are confusing, because we're talking about different entities. Do not confuse Social, User and SocialUser entities - all are different, and you do need to operate proper entities when you're trying to set them in setters. To have less WTF moments I'd suggest you to add typehints in your setters, like if you call setUser(), you only allow to pass User objects. If you call setUserSocial() - then only UserSocial are allowed. Typehints will help you a lot to avoid setting wrong object types.

And also, if you have cascade={persist} annotation - it won't mean to be saved to the DB unless you call flush. So, you would need to do any manipulations first, add/remove necessary objects and only then call flush(). But I'd recommend you to write a custom logic and call persist() explicitly on those entities you would like to save with flush() later. That annotation adds some magic and you can miss something important. So, doing it manually will be easier to understand I think.

Also, did you try to implement this way of complex manyToMany (i.e. oneToMany-ManyToOne) relation following this tutorial? Or did you start implementing this approach in your own project? I'd recommend you to follow the video exactly to implement the same things we show in this video. It will help you to understand the approach better. And then, when you successfully done what we show in this course, try to implement the same in your own project. But this time you should better understand what's going on there.

I hope this helps!

Cheers!

I'm getting a twig error when it hits the loop.
[2/2] Twig_Error_Runtime: An exception has been thrown during the rendering of a template ("Notice: Undefined index: genus").
{% for genusScientist in genus.scientists %}

I named my variable scientists instead of genusScientists but that shouldn't matter.

Any ideas?

Hey Sara,

So, somewhere you're calling "something.genus", and that "something" variable does not have a genus index. Please, notice the line where you have this error and try to debug, you can use "dump(something)" function for help.

Cheers!

Hmm, I can't see the doctrine's metadata, are you using yaml ?

I don't know why, but if I edit the post I see the whole code, but when published is not complete. Anyway, I'll link the code. It's simple php

https://pastebin.com/GvTSbfMv

Oh, it must be a thing from Disqus

I'm not really sure about this, but try writing the full namespace of your entities in your relationships, like:

   /**
     * @ORM\ManyToOne(targetEntity="AppBundle\Entity\Genus", inversedBy="genusScientists")
     * @ORM\JoinColumn(nullable=false)
     */
    private $genus;

Nope. I've also cleared the cache and the drop//create/migrate/load procedure, but I still got the error.
I'll give you also the Genus code, I don't know, maybe it's there.

https://pastebin.com/Eeybhgpy

Hmm, interesting... Try clearing doctrine's cache
bin/console doctrine:cache:clear-metadata

Can you show me the full error please ? (message + full stack trace)

Sure, in this text there is message + full trace + plain text version. I hope It will be useful

https://pastebin.com/i4jqLkMH

I think your problem is in your FormType, I will need to see the code too :)
Maybe you are not correctly setting up the class field option for your GenusScientist property

Allright, I'll link the whole code. Thank you :)

https://mega.nz/#!flQxVToD
key: !CaQhhmWzJnrVAVRw0x2qgkGP-1keUNowR8ZW5FMLhb0

Hey Josk

Finally I could track down your problem! :)
It is in your GenusFormType, you specified User class for your "genusScientists" field on Genus, just change it to "GenusScientist" class and everything should work (or at least you won't have that error again)

Cheers!

Boom! :)
The good news is that the error is gone, the bad news is that you were right when you said "that error again".
Here's the new one:

https://pastebin.com/BhRhWBx5

Sorry to bother again and again

Hey Jost!

Hahaha, don't worry, this time is much easier, you have a tiny problem, your method is called "getGenusScientists" with an "S" at the end, and Doctrine is expecting it without an "s", is a small problem for plural words, just remove the "s" and everything will work just fine

Cheers!

And we are back again to the same previous error "Given object is not an instance of the class this property was declared in
500 Internal Server Error - ReflectionException"
in vendor\doctrine\orm\lib\Doctrine\ORM\Mapping\ClassMetadataInfo.php at line 731 -
}
$id = $this->identifier[0];
$value = $this->reflFields[$id]->getValue($entity);
if (null === $value) {
return array();
at ReflectionProperty ->getValue (object(GenusScientist))
in vendor\doctrine\orm\lib\Doctrine\ORM\Mapping\ClassMetadataInfo.php at line 731 +
at ClassMetadataInfo ->getIdentifierValues (object(GenusScientist))
in vendor\symfony\symfony\src\Symfony\Bridge\Doctrine\Form\ChoiceList\IdReader.php at line 125

^_^

If the problem is the same, the fix will be almost the same :)

I see your query builder for 'genusScientists' field expects an UserRepository, instead of GenusScientistRepository, that might be the problem, or maybe any other field has defined a wrong type of class (like you had before)

Cheers!

I gave up :( I will take the finish folder and see the diff, It should work :)

Hey Josk!

Sorry for the late response, did it work ?
Uhh, we were just getting fun ;)

Cheers!

Yes, It did, but I can't tell exactly what was the problem: there were a lot of Scientist/Scientists mistake :)

Oh, I see. Well, I'm happy that you can keep going learning :)

Hi, in the end you wrote that should be no errors in the admin area. On admin/genus everything works but when I click on edit (admin/genus/13/edit) - huge error:

Type error: Argument 1 passed to AppBundle\Form\GenusFormType::AppBundle\Form\{closure}() must be an instance of AppBundle\Repository\UserRepository, instance of Doctrine\ORM\EntityRepository given

and it shows me:

in src\AppBundle\Form\GenusFormType.php (line 57)

->add('genusScientists', EntityType::class, [
'class' => GenusScientist::class,
'multiple' => true,
'expanded' => true,
'choice_label' => 'email',
'query_builder' => function (UserRepository $repo) {
return $repo->createIsScientistQueryBuilder();
}
])

What is wrong? I'm using Symfony 3.3.

Oh, and If I remove this part:

'query_builder' => function (UserRepository $repo) {
return $repo->createIsScientistQueryBuilder();
}

I get other error:

Neither the property "email" nor one of the methods "getEmail()", "email()", "isEmail()", "hasEmail()", "__get()" exist and have public access in class "AppBundle\Entity\GenusScientist".

**********

I noticed one more thing:

If I go to: users/{id}/edit - and choose ID which is a scientist I get error:

Given object is not an instance of the class this property was declared in

If choose ID which is not a scientist the form is showing. But when I check some checkboxes and click update I get the same error as above.

Please help. :)

Hey Krzysztof

You have a tiny error in your field definition. You specified GenusScientist as a class for the field `genusScientists`, when it should be the User class instead. Try again doing that change, and if that doesn't work, let us know!

Have a nice day :)

Hello, yes I had User but it gave me other error:

Given object is not an instance of the class this property was declared in.

Some other user (Josk, you can see below) had the same problem here and you wrote him this:

"It is in your GenusFormType, you specified User class for your "genusScientists" field on Genus, just change it to "GenusScientist" class and everything should work (or at least you won't have that error again)"

So I did it too...but this gave me error which I described in first my comment.

Ohh, forgive me, I meant `GenusScientist` class (instead of User class), when using a custom query builder you have to specify the same class as the relationship.
Have you tried doing it the "embedded" way? (Look at the next chapter)

If nothing works for you, can you upload your code to somewhere, so I can give it a quick look?

Cheers!

Thanks for quick reply. I did the next tutorial and editing genuses works. :) I still don't know why it didn't work earlier.

However I still have a problem which I mention in my other comment below. I mean this:

"If I go to: users/{id}/edit - and choose ID which is a scientist I get error:

Given object is not an instance of the class this property was declared in

If choose ID which is not a scientist the form is showing. But when I check some checkboxes and click update I get the same error as above."

Should it already works or maybe it will be fixed in next lessons?

If you want to look I uploaded my project here (without var and vendor folders): http://gryzli83.ayz.pl/aqua...

Hey Krzysztof, sorry for my late response!

I think I found your problem. Open your User class and look at your property studiedGenuses, it has a relationship with GenusScientist, everything good so far, but now, check at your UserEditForm, you specified that the studiedGenuses field is a Genus type, it should be the same as your relationship. Do that change and I believe everything will work just nice :)

Cheers!

Hi, I have a different message of error: Given object is not an instance of the class this property was declared in
500 Internal Server Error - ReflectionException.
The admin page has no error. Do you have some ideas?
Thanks

Hey Josk

Could you bring us some more information so we can help you ?

By doing what you get this error ?
Can you show me the full error message (with stack trace) ?
Which version of symfony are you using ?

Cheers!