Error Messages On Deletes (+new filenamesyntax code)

Hi,
I have been using Conquest for about a year with both Access and SQL Server. What a great product!
There is one problem that is consistent. When I delete a study or a patient from the "Browse Database" tab I get a messagebox "Row handle referred to a deleted row or a row marked for deletion". The delete does take place but the messageboxes keep coming up and I have to stop the app from the task manager.
What am I doing wrong?
Thanks
Tom

Hi Marcel,
We are using the code from your 14.12 alpha release. We modified the dgatefn.cpp file to place the dicom files in a custom directory structure. The same thing happens with 14.11 that we had not modified.

Could you tell me where the changes are that fix the delete bug? I would like to try those changes in our version.
Thanks
Tom

Thanks, I'll look through the code for references to the Bindfield.

What we did was add to the routine you have for the flexible filenamesyntax so we could also use date of study (year, month, day) and the center number (1st 2 digits of the PatientID). I'll be happy to send you the code if you are interested, it just adds some additional wild cards to the code you already have for the filenamesyntax in the INI file.
Tom

Hi Marcel,
Here is the code that we modified. Please understand that we are a bunch of vb.net programmers! If you tell me how to send you a big zip file I can send you a Visual Studio 2005 project that I use to compile this (I used your Linux code as a base).

I use these strings for the ini files:
#Regional Server
#FileNameSyntax = c%centernumber\Images\%centernumber\y_%year\m_%month\d_%day\%id\%studyuid_%seriesid_%imagenum_%time_%counter.dcm
#Center Server
#FileNameSyntax = %centernumber\y_%year\m_%month\d_%day\%id\%studyuid_%seriesid_%imagenum_%time_%counter.dcm

Here is the modified dgatefn.cpp code (moderator: condensed the code, will be included in next release).

/* MvH 19980326: changed file name generator to ensure correct sorting for avs_rdic */


.....


// Generate a file name prefix for an IOD


BOOL


GenerateFileName(
DICOMDataObject *DDOPtr,
char *Device,
char *Root,
char *filename,
BOOL NoKill,
int Syntax,
char *called,
char *calling,
Database *db)
{
VR *ImageNumber;


...


// Rick added this next line as an attempt to get the dicom information 
VR *StudyDate;


....


char dateStr [20];
char year[10];
char month[10];
char day[10];
char centerNumber[3];


.......


memset((void*)CSOPUID, 0, 65);
memset((void*)year, 0, 10);
memset((void*)month, 0, 10);
memset((void*)day, 0, 10);
memset((void*)centerNumber, 0, 3);
memset((void*)dateStr, 0, 20);
StudyDate = DDOPtr->GetVR(0x0008, 0x0020);
if(!StudyDate)
{
_strdate(dateStr);
}
else
{
strncpy(dateStr, (char*) StudyDate->Data, dgatefnmin(StudyDate->Length,20));
dateStr[StudyDate->Length] =0;
}


....


case 999: // configured filename: any string with %, e.g., %name_%id\%sopuid.v2


CID [16] = 0;
strncpy(day, dateStr+6, 2); day[2]=0;
strncpy(month, dateStr+4, 2); month[2]=0;
strncpy(year, dateStr, 4); year[4]=0;
strcpy(Root, CID);
strncpy(centerNumber, Root, 2); centerNumber[2]=0;
filename[0] = 0;
i = 0;
L = strlen(FileNameSyntaxString);


while (i<L)
{ if (FileNameSyntaxString[i] == '%')
{ i++;
  p = FileNameSyntaxString + i;
  if (memcmp(p, "name" , 4)==0) i+=4, strcat(filename, CName);
  else if (memcmp(p, "id" , 2)==0) i+=2, strcat(filename, CID);


  ......


  else if (memcmp(p, "callingae" , 9)==0) i+=9, strcat(filename, calling);
  else if (memcmp(p, "year" , 4)==0) i+=4, strcat(filename, year);
  else if (memcmp(p, "month" , 5)==0) i+=5, strcat(filename, month);
  else if (memcmp(p, "day" , 3)==0) i+=3, strcat(filename, day);
  else if (memcmp(p, "centernumber",12)==0) i+=12,strcat(filename, centerNumber);


// unrecognized names are not translated and appear as such in filename - good for debugging
}
else
{ l = strlen(filename);
filename[l] = FileNameSyntaxString[i++];
filename[l+1] = 0;
}
}


break;


default:


...

We are a small chain or Urgent Care centers. The centers each have a 2 digit center number which we use as a prefix to the patientID. We have a local conquest server at each center and a main conquest server at our regional office. I use windows replication to send the files from the centers to the main office and then use the dgate commands to import the files when they pop up in the directories.