get_status_display only returns the status not the LOAN_STATUS value about django-locallibrary-tutorial

[hamishwillee]

Hi @abdulwahidgul24085 ,
Thanks for this, and sorry for the delay getting back to you.

As @dlyrium say, this is not needed. The BookInstance model has a choices field named status. This includes a definition of the choices with a key and a human readable description.
The key is what is stored, and wherever a choices is used, Django automatically adds a function to get the human readable text. This is named get_FOO_display() where FOO is the name of the field (so in this case get_status_display()

This is not obvious at all, so I have updated the documentation on generic views

Closing now. Thanks - reporting the confusion is a useful contribution!

[hamishwillee]

Thanks very much. Can you please link to the tutorial page and the page in this repo that is affected. That will make this easier to review and update if needed.

[abdulwahidgul24085]

Sure. So in tutorial # 6 (Creating Generic List and Detail Views) in this output you can see the output shows the complete value for the LOAN_STATUS.

You can need to add the get_status_display function in your code since I see it talked about in the tutorial and does not exist in the BookInstance class in catalog/models.py
I hope this clarify's the confusion, and points to all the resources that need to be amended.

P.S. I hope you merge my code. It will make it my first official open-source contributed code. A man can dream can't he lol. Also, great tutorial. One of the most comprehensive introductions to Django that I could find.

[dlyrium]

get_FOO_display() is a method in the django.db.Models class. It should work fine, without any extra code.