#124 Fix #69 Display mac or ref when creating a new loan in the admin interface

Cool, it works !

But...

Firstly, you should rebase your work on master. (e.g. git pull -r origin master) and then repush your work so it can be merged automatically.

Secondly, label_from_instance does almost exactly the same as the function get_mac_or_serial from one of your previous PR. I think it would be better to refactor the code and use it.

get_mac_or_serial is a method of the class Loan (Loan.get_mac_or_serial), so it has to be moved to the class Item first (Item.get_mac_or_serial). Then, ItemChoiceField.label_from_instance could called it (obj.designation + ' ' + obj.get_mac_or_serial()).

You'll also need to update the method Loan.get_mac_or_serial to call self.item.get_mac_or_serial so that LoanAdmin can call Loan.get_mac_or_serial.

I hope it's clear enough :)

Humm ... Je sais pas trop. 'get_mac_or_serial' et 'label_from_instance' ne font pas exactement la même chose. Dans le cas ou la mac et le n° de série sont définis la première renvoie les deux alors que la seconde seulement la mac (je me disais que ça ferait trop pour la petite case sinon). Et comme bouger 'get_mac_or_serial' dans 'Item' implique de la redéfinir dans 'Loan' je me disais que ça ne simplifiait pas vraiment l'affaire ... T'en penses quoi ?

Il me semblait que c'était soit MAC soit numéro de série, je ne pensais pas qu'on pouvait avoir les deux.

Je pense qu'il vaut mieux être consistent et utiliser Item.get_mac_or_serial partout. Et, si on considère que c'est trop grand, soit modifier Item.get_mac_or_serial pour ne retourner que la MAC OU le serial, ou alors tronquer avant de l'utiliser dans ItemChoiceField.label_from_instance.

Ok, t'as gagné.

Great. Thank you.