Why m I getting this error in django urls?

Question

I am very new to Django and still trying to understand most things. l am currently working on a project where I need to add a details page for each link. But I am getting the following error. Error:

NoReverseMatch at /discover/

Reverse for 'disc-details' with arguments '('',)' not found. 1 pattern(s) tried: ['discover/(?P<discover_id>[0-9]+)/']

Following is my urls patterns for app:

url(r'discover/$', views.discover, name = 'discover'),
url(r'discover/(?P<discover_id>[0-9]+)/', views.disc_details, name = 'disc-details'),

and my views.py look something like:

def discover(request):
    disc = Discover.objects.all()
    return render(request, 'main/discover.html', {'disc': disc})

def disc_details(request, discover_id):
    dis = get_object_or_404(Discover, pk = discover_id)
    det = Discover.objects.get(pk = discover_id)
    return render(request, 'main/discDetails.html', {'det': det, 'dis': dis})

Whats happening here is discover is a html page containing various links and each link should have its on disc_details section. My html section has the following:

<a class="btn btn-success" href='{% url 'main:disc-details' dis.id%}' class = "detail-link">Details</a>

Help me fix this and please provide an explanation.

The error is in the discover.html, it looks like the the dis variable does not exists there. — willeM_ Van Onsem
– willeM_ Van Onsem, Commented Sep 27, 2018 at 9:46

Alasdair · Accepted Answer · 2018-09-27 09:59:43Z

1

Your context variable name is disc:

def discover(request):
    disc = Discover.objects.all()
    return render(request, 'main/discover.html', {'disc': disc})

So you should use disc in discover.html template also. Note you should iterate over disc values to get disc objects:

{% for el in disc %}
    '{% url 'main:disc-details' el.id %}'
{% endfor %}

edited Sep 27, 2018 at 9:59

Alasdair

310k59 gold badges605 silver badges534 bronze badges

answered Sep 27, 2018 at 9:47

neverwalkaloner

47.6k7 gold badges101 silver badges103 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Arjun Kashyap Over a year ago

It worked. Now can you explain this in simple language. (why are we looping ?)

neverwalkaloner Over a year ago

@ArjunKashyap we need to loop over disc since it contains multiple disc objects actually. Django's queryset Discover.objects.all() will give you list of discs. So you need to use loop to get access for each disc separately.

Arjun Kashyap Over a year ago

So I tried the loop method which worked when I added one entry in the database. Yet when I add more than one (say 2) it is showing two details (in a single section where there is supposed to be only one) link and when I click on them it takes me to the first database entry object. (Hope I made myself clear). I am adding a picture

Collectives™ on Stack Overflow

Why m I getting this error in django urls?

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related