Opened 10 years ago

Closed 10 years ago

#6857 closed (wontfix)

django.dispatch.robustapply does not work on jython

Reported by: Leo Soto M. Owned by: nobody
Component: Core (Other) Version: master
Severity: Keywords: jython pypy
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Currently, pydispatcher expect that every im_func object contains its corresponding func_code. That is not always the case, at least on Jython, raising AttributeError.

Attachments (3)

robustapply_jython_compatible.patch (1.8 KB) - added by Leo Soto M. 10 years ago.
robustapply_pypy_compatible.patch (1.3 KB) - added by anto.cuni@… 10 years ago. (1.8 KB) - added by Maciej Fijalkowski 10 years ago.
Test (py.test) for robustapply

Download all attachments as: .zip

Change History (8)

Changed 10 years ago by Leo Soto M.

comment:1 Changed 10 years ago by James Bennett

Triage Stage: UnreviewedDesign decision needed

We need to evaluate whether we're going to keep robust_apply in the dispatcher refactoring; if not this will be moot, but if so we'll need to take it into account.

comment:2 Changed 10 years ago by Keith Bussell

This would be fixed by #6814. As part of the clean-up/speed-up, robustapply has been removed.

Changed 10 years ago by anto.cuni@…

comment:3 Changed 10 years ago by anto.cuni@…

I've attached a new patch, that makes robustapply working on cpython, jython and pypy at the same time.

Changed 10 years ago by Maciej Fijalkowski

Attachment: added

Test (py.test) for robustapply

comment:4 Changed 10 years ago by Maciej Fijalkowski

Keywords: pypy added

I added a test (py.test style, I can move it to unittest style if anyone wants) to prove that robustapply behaves exactly the same everywhere.

comment:5 Changed 10 years ago by Gary Wilson

Resolution: wontfix
Status: newclosed

#6814 is likely to make it into 1.0 and it gets rid of the slow robust apply stuff. Let's focus our efforts there.

Note: See TracTickets for help on using tickets.
Back to Top