Re: DBIx::ContextualFetch trouble

[prev] [thread] [next] [Date index for 2004/12/27]

  This message is in MIME format.  The first part should be readable text,
  while the remaining parts are likely unreadable without MIME-aware tools.

--8323328-1698267270-1104159288=:15017
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed

On Mon, 27 Dec 2004, Tony Bowden wrote:

> On Mon, Dec 27, 2004 at 04:39:27PM +0200, Gabor Szabo wrote:
>> It seems to me that sometimes the value passed to execute()
>> had an effect and sometimes it did not have an effect on the outcome.
>> (Talking only abot cases with a ? placeholder)
>> If you are still interested I can send you the test file in one piece.
>
> I'm confused.
>
> As I understand it, the problem with DBIx::CF was with the test calling
> execute twice.


That was only one problem.
There was a whole set of other cases where the behavor was strange.
I am attaching my test script (see the comments as well)

Looking at the new test script you put in the distro now I understand
that I might have misunderstood the API.

It seems to be strange that sometimes you call ->execute manually
and sometimes you leave it to the module to do it. This can cause
someone to accidently call execute twice and you don't even get
a warning from DBIx::ContextualFetch.
(just from some other module and even that only I get :)


Gabor

--8323328-1698267270-1104159288=:15017
Content-Type: APPLICATION/x-troff; name="01.t"
Content-Transfer-Encoding: BASE64
Content-ID: <Pine.LNX.4.61.0412271654480.15017@xxxxx.xxx.xx.xx>
Content-Description: 
Content-Disposition: attachment; filename="01.t"

IyEvdXNyL2Jpbi9wZXJsIC13Cgp1c2Ugc3RyaWN0OwoKdXNlIFRlc3Q6Ok1v
cmU7CgpCRUdJTiB7CglldmFsICJ1c2UgREJEOjpTUUxpdGUiOwoJcGxhbiAk
QCA/IChza2lwX2FsbCA9PiAnbmVlZHMgREJEOjpTUUxpdGUgZm9yIHRlc3Rp
bmcnKSA6ICh0ZXN0cyA9PiAzMCk7Cn0KCnVzZSBGaWxlOjpUZW1wIHF3L3Rl
bXBmaWxlLzsKbXkgKHVuZGVmLCAkREIpID0gdGVtcGZpbGUoKTsKbXkgQERT
TiA9ICgKCSJkYmk6U1FMaXRlOmRibmFtZT0kREIiLCAnJywgJycsCgl7IEF1
dG9Db21taXQgPT4gMSwgUm9vdENsYXNzID0+ICJEQkl4OjpDb250ZXh0dWFs
RmV0Y2giIH0pOwoKbXkgJGRiaCA9IERCSS0+Y29ubmVjdChARFNOKTsKCiRk
YmgtPmRvKCJDUkVBVEUgVEFCTEUgZm9vIChpZCBJTlRFR0VSLCBuYW1lIFRF
WFQpIik7Cm15ICRpbnNlcnQgPSAkZGJoLT5wcmVwYXJlKCJJTlNFUlQgSU5U
TyBmb28gVkFMVUVTICg/LCA/KSIpOwokaW5zZXJ0LT5leGVjdXRlKDEsICJG
cmVkIik7CiRpbnNlcnQtPmV4ZWN1dGUoMiwgIkJhcm5leSIpOwoKc3ViIG1h
a2Vfc3RoIHsKCW15ICRzcWwgPSBzaGlmdDsKCShteSAkc3RoID0gJGRiaC0+
cHJlcGFyZSgkc3FsKSktPmV4ZWN1dGUoQF8pOwoJcmV0dXJuICRzdGg7Cn0K
CnsgICAgIyBmZXRjaAoJbXkgJHN0aCAgPSBtYWtlX3N0aCgiU0VMRUNUICog
RlJPTSBmb28gT1JERVIgQlkgaWQiKTsKCW15IEBnb3QxID0gJHN0aC0+ZmV0
Y2g7CglpcyAkZ290MVsxXSwgIkZyZWQiLCAnZmV0Y2ggQCc7CglteSAkZ290
MiA9ICRzdGgtPmZldGNoOwoJaXMgJGdvdDItPlsxXSwgIkJhcm5leSIsICdm
ZXRjaCAkJzsKfQoKeyAgICAjIEZldGNoIEhhc2gKCW15ICRzdGggID0gbWFr
ZV9zdGgoIlNFTEVDVCAqIEZST00gZm9vIE9SREVSIEJZIGlkIik7CglteSAl
Z290MSA9ICRzdGgtPmZldGNoX2hhc2g7CglpcyAkZ290MXtuYW1lfSwgIkZy
ZWQiLCAnZmV0Y2hfaGFzaCAlJzsKCW15ICRnb3QyID0gJHN0aC0+ZmV0Y2hf
aGFzaDsKCWlzICRnb3QyLT57bmFtZX0sICJCYXJuZXkiLCAnZmV0Y2hfaGFz
aCAkJzsKCW15ICVnb3QzID0gZXZhbCB7ICRzdGgtPmZldGNoX2hhc2ggfTsK
CWlzIGtleXMgJWdvdDMsIDAsICJOb3RoaW5nIGF0IHRoZSBlbmQiOwoJaXMg
JEAsICIiLCAiQW5kIG5vIGVycm9yIjsKfQoKeyAgICAjIGZldGNoYWxsIEAK
CW15ICRzdGggPSBtYWtlX3N0aCgiU0VMRUNUICogRlJPTSBmb28gT1JERVIg
QlkgaWQiKTsKCW15IEBnb3QgPSAkc3RoLT5mZXRjaGFsbDsKCWlzICRnb3Rb
MV0tPlsxXSwgIkJhcm5leSIsICdmZXRjaGFsbCBAJzsKCWlzIEBnb3QsIDIs
ICdmZXRjaGFsbCBAIGNvdW50JzsKfQoKeyAgICAjIGZldGNoYWxsIEAKCW15
ICRzdGggPSBtYWtlX3N0aCgiU0VMRUNUICogRlJPTSBmb28gV0hFUkUgaWQg
PiA/IE9SREVSIEJZIGlkIiwgMCk7CglteSBAZ290ID0gJHN0aC0+ZmV0Y2hh
bGwoMCk7ICMgcGFyYW1ldGVyIGlycmVsZXZhbnQgaGVyZQoJaXMgQGdvdCwg
MiwgJ2ZldGNoYWxsIEAgY291bnQnOwp9Cgp7ICAgICMgZmV0Y2hhbGwgQAoJ
bXkgJHN0aCA9IG1ha2Vfc3RoKCJTRUxFQ1QgKiBGUk9NIGZvbyBXSEVSRSBp
ZCA+ID8gT1JERVIgQlkgaWQiLCAwKTsKCW15IEBnb3QgPSAkc3RoLT5mZXRj
aGFsbCgxMDApOyAjIHBhcmFtZXRlciBpcnJlbGV2YW50IGhlcmUKCWlzIEBn
b3QsIDIsICdmZXRjaGFsbCBAIGNvdW50JzsKfQoKeyAgICAjIGZldGNoYWxs
IEAKCW15ICRzdGggPSBtYWtlX3N0aCgiU0VMRUNUICogRlJPTSBmb28gV0hF
UkUgaWQgPiA/IE9SREVSIEJZIGlkIiwgMSk7CglteSBAZ290ID0gJHN0aC0+
ZmV0Y2hhbGwoMCk7ICMgcGFyYW1ldGVyIGlycmVsZXZhbnQgaGVyZQoJaXMg
QGdvdCwgMSwgJ2ZldGNoYWxsIEAgY291bnQnOwp9Cgp7ICAgICMgZmV0Y2hh
bGwgQAoJbXkgJHN0aCA9IG1ha2Vfc3RoKCJTRUxFQ1QgKiBGUk9NIGZvbyBX
SEVSRSBpZCA+ID8gT1JERVIgQlkgaWQiLCAxKTsKCW15IEBnb3QgPSAkc3Ro
LT5mZXRjaGFsbCgxKTsgIyBwYXJhbWV0ZXIgaXJyZWxldmFudCBoZXJlCglp
cyBAZ290LCAxLCAnZmV0Y2hhbGwgQCBjb3VudCc7Cn0KCnsgICAgIyBmZXRj
aGFsbCBACglteSAkc3RoID0gbWFrZV9zdGgoIlNFTEVDVCAqIEZST00gZm9v
IFdIRVJFIGlkID4gPyBPUkRFUiBCWSBpZCIsIDIpOwoJbXkgQGdvdCA9ICRz
dGgtPmZldGNoYWxsKDEpOyAjIHBhcmFtZXRlciBpcnJlbGV2YW50IGhlcmUK
CWlzIEBnb3QsIDAsICdmZXRjaGFsbCBAIGNvdW50JzsKfQoKeyAgICAjIGZl
dGNoYWxsICQKCW15ICRzdGggPSBtYWtlX3N0aCgiU0VMRUNUICogRlJPTSBm
b28gT1JERVIgQlkgaWQiKTsKCW15ICRnb3QgPSAkc3RoLT5mZXRjaGFsbDsK
CWlzICRnb3QtPlsxXS0+WzFdLCAiQmFybmV5IiwgJ2ZldGNoYWxsICQnOwp9
Cgp7ICAgICMgZmV0Y2hhbGxfaGFzaCBACglteSAkc3RoID0gbWFrZV9zdGgo
IlNFTEVDVCAqIEZST00gZm9vIE9SREVSIEJZIGlkIik7CglteSBAZ290ID0g
JHN0aC0+ZmV0Y2hhbGxfaGFzaDsKCWlzICRnb3RbMV0tPntuYW1lfSwgIkJh
cm5leSIsICdmZXRjaGFsbF9oYXNoIEAnOwp9Cgp7ICAgICMgZmV0Y2hhbGxf
aGFzaCAkCglteSAkc3RoID0gbWFrZV9zdGgoIlNFTEVDVCAqIEZST00gZm9v
IE9SREVSIEJZIGlkIik7CglteSAkZ290ID0gJHN0aC0+ZmV0Y2hhbGxfaGFz
aDsKCWlzICRnb3QtPlsxXS0+e25hbWV9LCAiQmFybmV5IiwgJ2ZldGNoYWxs
X2hhc2ggQCc7Cn0KCnsgICAgIyBzZWxlY3Rfcm93CglteSAkc3RoID0gbWFr
ZV9zdGgoIlNFTEVDVCAqIEZST00gZm9vIFdIRVJFIGlkID0gPyIsIDEpOwoJ
bXkgKCRpZCwgJG5hbWUpID0gJHN0aC0+c2VsZWN0X3JvdygxKTsKCWlzICRu
YW1lLCAiRnJlZCIsICJzZWxlY3Rfcm93IjsKfQoKeyAgICAjIHNlbGVjdF9j
b2wKCW15ICRzdGggICA9IG1ha2Vfc3RoKCJTRUxFQ1QgbmFtZSBGUk9NIGZv
byBXSEVSRSBpZCA+ID8gT1JERVIgQlkgaWQiLCAwKTsgIyBpcyBwYXJhbWV0
ZXIgaXJyZWxldmFudCA/CglteSBAbmFtZXMgPSAkc3RoLT5zZWxlY3RfY29s
KDApOwoJaXMgJG5hbWVzWzBdLCAiRnJlZCIsICJzZWxlY3RfY29sIjsKCWlz
ICRuYW1lc1sxXSwgIkJhcm5leSIsICJzZWxlY3RfY29sIjsKCWlzIEBuYW1l
cywgMiwgJz4gMCBjb3VudCc7Cn0KCnsgICAgIyBzZWxlY3RfY29sCglteSAk
c3RoICAgPSBtYWtlX3N0aCgiU0VMRUNUIG5hbWUgRlJPTSBmb28gV0hFUkUg
aWQgPiA/IE9SREVSIEJZIGlkIiwgMCk7ICMgaXMgcGFyYW1ldGVyIGlycmVs
ZXZhbnQgPwoJbXkgQG5hbWVzID0gJHN0aC0+c2VsZWN0X2NvbCgxKTsKCWlz
ICRuYW1lc1swXSwgIkJhcm5leSIsICJzZWxlY3RfY29sIjsKCWlzIEBuYW1l
cywgMSwgJz4gMCBjb3VudCc7Cn0KCnsgICAgIyBzZWxlY3RfY29sCglteSAk
c3RoICAgPSBtYWtlX3N0aCgiU0VMRUNUIG5hbWUgRlJPTSBmb28gV0hFUkUg
aWQgPiA/IE9SREVSIEJZIGlkIiwgMTAwKTsgIyBpcyBwYXJhbWV0ZXIgaXJy
ZWxldmFudCA/CglteSBAbmFtZXMgPSAkc3RoLT5zZWxlY3RfY29sKDEpOwoJ
aXMgJG5hbWVzWzBdLCAiQmFybmV5IiwgInNlbGVjdF9jb2wiOwoJaXMgQG5h
bWVzLCAxLCAnPiAxIGNvdW50JzsKfQoKCnsgICAgIyBzZWxlY3RfdmFsCglt
eSAkc3RoICA9IG1ha2Vfc3RoKCJTRUxFQ1QgbmFtZSBGUk9NIGZvbyB3aGVy
ZSBpZCA9ID8iLCAxKTsgIyBpcyBwYXJhbWV0ZXIgaXJyZWxldmFudCA/Cglt
eSAkbmFtZSA9ICRzdGgtPnNlbGVjdF92YWwoMSk7CglpcyAkbmFtZSwgIkZy
ZWQiLCAic2VsZWN0X3ZhbCI7CglteSAkbm9uYW1lID0gJHN0aC0+c2VsZWN0
X3ZhbCgyKTsKCWlzICRub25hbWUsICJCYXJuZXkiLCAic2VsZWN0X3ZhbCB3
aGVyZSBpZD0xIjsKfQoKeyAgICAjIEV4ZWN1dGUgYmluZGluZwoJbXkgJHN0
aCA9ICRkYmgtPnByZXBhcmUoIlNFTEVDVCAqIEZST00gZm9vIFdIRVJFIGlk
ID4gPyBPUkRFUiBCWSBpZCIpOwoJJHN0aC0+ZXhlY3V0ZShbMF0sIFsgXG15
ICgkaWQsICRuYW1lKSBdKTsKCSRzdGgtPmZldGNoOwoJaXMgJGlkLCAgIDEs
ICAgICAgImJvdW5kIGlkIDEiOwoJaXMgJG5hbWUsICJGcmVkIiwgIm5hbWUg
PSBGcmVkIjsKCSRzdGgtPmZldGNoOwoJaXMgJGlkLCAgIDIsICAgICAgICAi
Ym91bmQgaWQgMiI7CglpcyAkbmFtZSwgIkJhcm5leSIsICJuYW1lID0gQmFy
bmV5IjsKfQoKIyBUaGUgd2FybmluZyB3ZSBnZXQgd2hlbiB3ZSBwcmVwYXJl
IGFuIFNRTCB3aXRoIGEgcGxhY2UgaG9sZGVyIGJ1dCB0aGVuIGV4ZWN1dGUg
aXQKIyB3aXRob3V0IGEgcGFyYW1ldGVyIGxvb2tzIGxpa2UgdGhpczoKIyBx
ci9Vc2Ugb2YgdW5pbml0aWFsaXplZCB2YWx1ZSBpbiBzdWJyb3V0aW5lIGVu
dHJ5LzsKIyBQcm9iYWJseSBEQkl4OjpDb250ZXh0RmV0Y2ggc2hvdWxkIGRp
ZSBpbiBzdWNoIGNhc2VzLgojIFRoZSB3YXJuaW5nIGFycml2ZXMgZnJvbSB0
aGUgCiMgICAgICAgIF91bnRhaW50X2V4ZWN1dGUgc3Vicm91dGluZSAKIyBm
cm9tIHRoZSAKIyAgICAgICBteSAkcmV0ID0gJHN0aC0+U1VQRVI6OmV4ZWN1
dGUoQF8pOwojIGxpbmUK

--8323328-1698267270-1104159288=:15017--