[bug_fixed] bug report: read blob in ADO

Code patches written by our users to solve certain "problems" that were not solved, yet.

Moderators: gto, cipto_kh, EgonHugeist, mdaems

Post Reply
zx
Fresh Boarder
Fresh Boarder
Posts: 16
Joined: 13.10.2005, 14:05

[bug_fixed] bug report: read blob in ADO

Post by zx »

greetings. I've been sending a number of patches to Zeos team since 6.1.5 or 6.1.4 version, but as member rotations took place, this patches were not taken a good look at..

so I'm trying to send 'em one more time.

bug report: read blob in ADO

[platform notes]
zeosdbo-6.6.1-beta
protocol: ado
delphi7
Microsoft SQL server 2000
field kind = image

[problem]
when you fetch blob field from database with zeosdbo components
using ADO protocol the last byte of blob field is lost.

[suggested solution]
======== [dbc\ZDbcAdoResultSet.pas] ========
{
(scope)
function TZAdoResultSet.GetBlob(ColumnIndex: Integer): IZBlob;
(origin)
{
Result := TZAbstractBlob.CreateWithData(P, VarArrayHighBound(V, 1));
}
(changed into)
{
Result := TZAbstractBlob.CreateWithData(P, VarArrayHighBound(V, 1) + 1);
}
(purpose)
{to fix error: read blob truncates its last byte}
}
User avatar
mdaems
Zeos Project Manager
Zeos Project Manager
Posts: 2766
Joined: 20.09.2005, 15:28
Location: Brussels, Belgium
Contact:

Post by mdaems »

Hi zx,

Can you please try to get the test suite installed and working on your machine? We don't have people using ADO, so it would be nice if you could tell us these patches don't make it all worse. I suppose some tests already fail from the beginning, but if you count the numbers 'before' and 'after' changes we know if the score goes up or down.

As you see, this time we don't forget to have a look at your patches, but as we don't have a lot of ADO users and no developers we have some quality assurance problem.

This patch is committed in rev. 253. (Testing branch)

Mark
zx
Fresh Boarder
Fresh Boarder
Posts: 16
Joined: 13.10.2005, 14:05

Post by zx »

mdaems wrote:Can you please try to get the test suite installed and working on your machine?
I'll try to do my best to make it so. But now I do not really have time to do it.
mdaems wrote: As you see, this time we don't forget to have a look at your patches
Thanx a lot. I said about all that misunderstandings only because I am sending this stuff for the third if not forth time (first time - to your team).
User avatar
mdaems
Zeos Project Manager
Zeos Project Manager
Posts: 2766
Joined: 20.09.2005, 15:28
Location: Brussels, Belgium
Contact:

Post by mdaems »

Sorry we had to drop the sourceforge trackers. I suppose it was there you posted previously. It was a real mess when we had to take over. Nobody knew what was done or not done in the source version we had. So we had to start over issue collection. At first it worked fine using the bugs forum but that became to complicated with all bugs standing there closed and open together. So we decided to move to the bug tracker.

Mark
Post Reply