search

Bacon / BaconUser

BaconUser provides simple user management
BSD 2-Clause "Simplified" License
14 stars 9 forks source link

Use simple id for entities #24

Closed bakura10 closed 11 years ago

bakura10 commented 11 years ago

This is rebased against latest master:

"Currently, custom name for identifiers were set (user_id…). I suggest don't change the default because it brings zero advantage, but rather a very annoying disadvantage.

When doing associations (especially with something as common as User), Doctrine assumes that the identifier has the key "id". If it's not the case (as it is currently), you need to manually set a JoinColumn so that the association is correct. This is very annoying, and can be forgotten easily.

So I suggest having the easy way and leave the default here."

coveralls commented 11 years ago

Coverage Status

Coverage remained the same when pulling 86f36ff42a6f185a8ecd844086291ccd699e790c on bakura10:id into 4895b2f4fd9e06febb892c3979ffa8a5a7cdbbfe on Bacon:master.