Issue1103

classification
Title: __tojava__ should work the same way on old-style and new-style classes
Type: Severity: normal
Components: Core Versions:
Milestone:
process
Status: open Resolution: remind
Dependencies: Superseder:
Assigned To: nobody Nosy List: amak, fwierzbicki, leosoto, nobody, zyasoft
Priority: low Keywords:

Created on 2008-08-11.21:59:40 by leosoto, last changed 2014-09-26.05:49:18 by zyasoft.

Messages
msg3423 (view) Author: Leonardo Soto (leosoto) Date: 2008-08-11.21:59:40
I added __tojava__ to *Derived, without knowing that it was already
supported on PyInstance. So now we have two different (and not exactly
compatible) implementations for the same idea.

The problem is that the PyInstance.__tojava__ didn't let the
python-level __tojava__ a chance when the requested class was
Object.class. I don't like this (and will break zxJDBC when datetime.*
or decimal.Decimal parameters are passed without explicitly setting the
types as an extra argument).
msg6869 (view) Author: Alan Kennedy (amak) Date: 2012-03-19.20:44:38
Leo, there's been a lot of changes in the last few years. Is this issue still present?

Can it be resolved?
msg8525 (view) Author: Jim Baker (zyasoft) Date: 2014-05-22.00:47:51
Should revisit.
msg9046 (view) Author: Jim Baker (zyasoft) Date: 2014-09-26.05:49:18
So right now bonus functionality for using new style classes. This is probably OK given how easy it is to convert.

Doc fix somewhere is probably the right solution.
History
Date User Action Args
2014-09-26 05:49:18zyasoftsetpriority: high -> low
assignee: fwierzbicki -> nobody
messages: + msg9046
nosy: + nobody
2014-05-22 00:47:51zyasoftsetnosy: + zyasoft
messages: + msg8525
2013-02-27 17:57:14fwierzbickisetpriority: normal -> high
assignee: leosoto -> fwierzbicki
resolution: remind
2012-03-19 20:44:38amaksetnosy: + amak
messages: + msg6869
2009-03-14 14:25:12fwierzbickisetpriority: normal
2008-11-01 13:56:42fwierzbickisetnosy: + fwierzbicki
components: + Core
2008-08-11 21:59:51leosotosetassignee: leosoto
2008-08-11 21:59:40leosotocreate