From b4182316afdf4897d273bc672be014df0c4f71a3 Mon Sep 17 00:00:00 2001
From: helmutm <helmutm@fd906abe-77d9-0310-91a1-e0d9ade77398>
Date: Fri, 27 Jan 2006 09:14:06 +0000
Subject: [PATCH] Refactored ordered container stuff out of loops package and
 put into cybertools.container

git-svn-id: svn://svn.cy55.de/Zope3/src/cybertools/trunk@1040 fd906abe-77d9-0310-91a1-e0d9ade77398
---
 container/contents.pt | 1 -
 container/ordered.py  | 7 ++++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/container/contents.pt b/container/contents.pt
index c386fc2..15d0d4d 100644
--- a/container/contents.pt
+++ b/container/contents.pt
@@ -5,7 +5,6 @@
   <div metal:define-macro="contents">
 
     <tal:checkmove define="dummy view/checkMoveAction">
-
     <form name="containerContentsForm" method="post" action="."
           tal:attributes="action request/URL"
           tal:define="container_contents view/listContentInfo">
diff --git a/container/ordered.py b/container/ordered.py
index bdd2db8..85070da 100644
--- a/container/ordered.py
+++ b/container/ordered.py
@@ -26,6 +26,7 @@ from zope.app import zapi
 from zope.app.container.ordered import OrderedContainer as BaseOrderedContainer
 from zope.cachedescriptors.property import Lazy
 from zope.app.container.browser.contents import JustContents
+from zope.app.i18n import ZopeMessageFactory as _
 
 
 class OrderedContainerView(JustContents):
@@ -44,9 +45,9 @@ class OrderedContainerView(JustContents):
                 delta = request.get('delta', 1)
                 ids = request.get('ids', [])
                 if ids:
-                    m = getattr(self, var, None)
-                    if m:
-                        m(ids, delta)
+                    getattr(self, var, None)(ids, delta)
+                else:
+                    self.error = _("You didn't specify any ids to move.")
                 return
 
     def move_down(self, ids=[], delta=1):