Как сделать переадресацию из DetailView в ListView?

Question

[emeraldsinner]

Есть ListView, в нем список обьектов и ссылки на них в DetailView, ListView не принимает никаких параметров, DetailView принимает id обьекта. В DetailView есть ссылка на себя с передачей параметра в функцию, нужно чтоб в функции была переадресация обратно в ListView.
urls.py:

url(r'^manage_orders/$', manage_orders, name='manage_orders'),
    url(r'^order_details/(?P<pk>\d+)$', order_details, name='order_details'),

DetailView функция:

try:
            accept = self.request.GET.get('accept')
        except:
            accept = False
if accept:
            Orders.objects.filter(id=self.kwargs['pk'], organization=self.request.user.organization).update(is_order=True, status='accepted')
            #return HttpResponseRedirect('/manage_orders')

и ссилка в темплейте:

<a href="{% url 'order_details' order.id %}?accept=True"><button>Accept</button></a>

Если использую HttpResponseRedirect или reverse_lazy дает ошибку :

Reverse for 'order_details' with arguments '('',)' and keyword arguments '{}' not found. 1 pattern(s) tried: ['order_details/(?P\\d+)$']

Comments

[Ростислав Григорьев]

Пример использования HttpResponseRedirect не указан. А ведь проблема скорее всего в неправильном вызове функции reverse.

[emeraldsinner]

Ростислав Григорьев: использую просто: return HttpResponseRedirect('/manage_orders')

Answer 1

[Ростислав Григорьев]

{% url 'order_details' pk=order.id %}

Comments

[emeraldsinner]

Не изменилось ничего: Reverse for 'order_details' with arguments '()' and keyword arguments '{u'pk': ''}' not found. 1 pattern(s) tried: ['order_details/(?P\\d+)$']

[Ростислав Григорьев]

А чему равен order.id ?

[emeraldsinner]

Ростислав Григорьев: id обьекта, в моем случае для теста : 5

[Ростислав Григорьев]

Так ты уверен что order.id выдает именно 5? А не пустую строку или None?

[emeraldsinner]

Ростислав Григорьев: да, функция делает апдейт обьекта нормально, значит pk передает, в self.kwargs['pk']

[Ростислав Григорьев]

kwargs['pk'] и order.id все же разные объекты. Лучше убедится прямо в шаблоне, убрать баговый урл и прописать ->>{{ order.id }}<--
Тогда должно стать ясно

[Ростислав Григорьев]

Я уверен в том, что во вьюхе просто не передаются в шаблон order.
Проверить, что урл сработает можно указав напрямую число {% url 'order_details' pk=5 %}

[emeraldsinner]

Ростислав Григорьев: Поставил a href="{% url 'order_details' pk=5 %}"

[Ростислав Григорьев]

emeraldsinner: и?
теперь нужно отправить нормально в шаблон order, и когда это будет сделано прописать туда как было order.id

[emeraldsinner]

Ростислав Григорьев: если пишу так: return reverse_lazy('manage_orders', kwargs = {'pk' : self.kwargs['pk'], }), то уже ругается на ListView: Reverse for 'manage_orders' with arguments '()' and keyword arguments '{'pk': u'5'}' not found. 1 pattern(s) tried: ['manage_orders/$']

[Ростислав Григорьев]

emeraldsinner: так ведь у manage_orders и нет в урл айди. есть только у order_details ...

[emeraldsinner]

Ростислав Григорьев: В етом и проблема была с самого начала. Как сделать переадресацию со вьюхи, где есть айди на вьюху где ее нет

[Ростислав Григорьев]

> Как сделать переадресацию со вьюхи, где есть айди на вьюху где ее нет

шта?? переформулируй вопрос

[emeraldsinner]

с order_details, которое принимает pk, при нажимании на кнопку вызывается get_context_data, нужно чтоб с get_context_data шла переадресация на manage_orders, которое не принимает pk

[Ростислав Григорьев]

И в чем проблема-то? reverse('manage_orders') даст необходимую ссылку "без pk"

[emeraldsinner]

Ростислав Григорьев: Если return reverse('manage_orders') то таже пробоема Reverse for 'order_details' with arguments '()' and keyword arguments '{u'pk': ''}' not found. 1 pattern(s) tried: ['order_details/(?P\\d+)$']

[Ростислав Григорьев]

При чем здесь order_details когда урл manage_orders.
Ошибка, наверное, опять в шаблоне же

[emeraldsinner]

Ростислав Григорьев: Вот весь шаблон:

{% extends "header.html" %}
{% load staticfiles %}
{% load custom_tag %}

{% block body_block %}

<ul>
	<li><b>Title: </b>{{ order.title }}</li>
	<li><b>Profile: </b>{{ order.profile }}</li>
	<li><b>Create date: </b>{{ order.created }}</li>
	<li><b>Update date: </b>{{ order.updated }}</li>
	<li><b>Status: </b>{{ order.status }}</li>
	<li><b>Products: </b>
		<table border='1'>
			<tr>
				<th>Group name</th>
				<th>Art</th>
				<th>Product name</th>
				<th>Image</th>
            	<th>Packing</th>
            	<th>Cost</th>
            	<th>Discount price</th>
            	<th>Count</th>
            	<th>Sum</th>
			</tr>
			{% for product in orders.orderproducts_set.all %}
				<tr>
					<td>{{ product.product.group }}</td>
					<td>{{ product.product.art }}</td>
					<td>{{ product.product.product_name }}</td>
					<td><img src="{{ MEDIA_URL }}{{product.product.photo}}" height="50" width="50"></td>
              		<td>{{product.product.get_packing_display}}</td>
	              	<td>{{product.product.cost}}</td>
	              	<td>{{product.price}}</td>
	              	<td>{{product.count}}</td>
	              	<td>{{product.price|divide:product.count}}</td>
				</tr>
			{% endfor %}
		</table>
	</li>
</ul>
{% if order.status != "accepted" %}
<a href="{% url 'order_details' pk=order.id %}?accept=True"><button>Accept</button></a>
{% endif %}
<a href="{% url 'order_details' pk=order.id %}?decline=True"><button>Decline</button></a>

{% endblock %}

И вся вьюха:

class OrderDetailsView(DetailView):
    model = Orders
    template_name = 'orders/order_details.html'

    def get_context_data(self, *args, **kwargs):
        try:
            accept = self.request.GET.get('accept')
        except:
            accept = False
        try:
            decline = self.request.GET.get('decline')
        except:
            decline = False
        context = super(OrderDetailsView, self).get_context_data(*args, **kwargs)
        context['order'] = Orders.objects.get(id=self.kwargs['pk'], organization=self.request.user.organization)
        if accept:
            Orders.objects.filter(id=self.kwargs['pk'], organization=self.request.user.organization).update(is_order=True, status='accepted')
            return reverse('manage_orders')
        if decline:
            Orders.objects.filter(id=self.kwargs['pk'], organization=self.request.user.organization).update(is_order=False, status='declined')
            return reverse('manage_orders')
        return context

[Ростислав Григорьев]

В методе get_context_data не нужно передавать return reverse('manage_orders')

Это делается в методе dispatch, get или post. Причем возвращать нужно не строку урл, а респонс redirect(reverse('manage_orders'))

[emeraldsinner]

Ростислав Григорьев: Спасибо, переделал кнопки под формы, которые отправляют post запрос, написал функцию post и в ней сделал респонс.

[Никита Котенко]

Ростислав Григорьев: emeraldsinner: ребята, у меня тоже проблема, но менее тривиальная: —при рендере админского шаблона такая ошибка
Reverse for 'send_prepay' with arguments '(123,)' and keyword arguments '{}' not found. 0 pattern(s) tried: []

ссылка в шаблоне с параметром 123 для теста:
{% url 'send_prepay' 123 %}
------------ Модель админки -------------
class ReservationAdmin(admin.ModelAdmin):
...
def get_urls(self):
urls = super(ReservationAdmin, self).get_urls()
urlpatterns = [url(r'^(\d+)/send_prepay/$', self.send_prepay, name="send_prepay")]
return urlpatterns + urls

def send_prepay(self, request, reservation_id):
response = HttpResponse(reservation_id)
return response
------------ Модель админки -------------

[Никита Котенко]

сам send_prepay правильно отдает страницу с параметром. Проблема что в шаблоне не происходит вызов паттерна

[Никита Котенко]

сам решил. нужно было указать пространство имен {% url 'admin:send_prepay' 123 %}