can change password

[r][m]

My exception logs, if needed.
I had none for delete account.
Mine are wine/linux.

[SilentPliz]

@r][m

Delete accounts is OK for me too with linux.

This only occurs on Windows XP when I try to delete a list of accounts, one by one... and it's pretty random.

[r][m]

@ SilentPliz
Thanks for the help and confirming this.

Guess solving this is up to the Boss now? 

[rejetto]

i'm sorry guys, but these logs miss some information.
Maybe it's because of Wine.
if you re-do it with the official #276 i may be able to make use of the numbers.

[SilentPliz]

Voila!

I'm sick, I'll go to bed!


Edit: created with the current build 276 decompressed with UPX

[r][m]

And mine with 276

[rejetto]

for some straneg reason r][m's logs are much shorter, but the error is the same.
and even it's the same between "add" and "rename".
Indeed, add is just a "create bulk" + "rename".
I can't fix this problem, so i will try a workaround.
Let me know if it works: http://www.sendspace.com/file/vww8ah


SP and programmers may be interested in knowing that the line issuing the error is not in HFS' code but in delphi's one.
After renaming, this line in HFS is executed
old:=@tempAccounts[item.Index].user;

reading item.index, and reaching the problem in method getIndex() at line
  if Owner.Owner.OwnerData then

here, apparently, a reference/pointer is wrong under Wine, while it's always correct on native Windows. It's about a chain of ownership, the item belongs to a list of items, belonging to the widget in turn.

[SilentPliz]

for some straneg reason r][m's logs are much shorter, but the error is the same.
and even it's the same between "add" and "rename".
Indeed, add is just a "create bulk" + "rename".
I can't fix this problem, so i will try a workaround.
Let me know if it works: http://www.sendspace.com/file/vww8ah

My file was longer, I had "unzipped" the executable before using it.
-----------

Your new solution does not work better ... always the same bugs.

Here too I have "unzipped" hfs.exe for more informations

[Mars]

@rejetto
What do you think about to change 'old' defined as pointer to string, as STRING

procedure ToptionsFrm.accountsBoxEdited(Sender: TObject; Item: TListItem; var S: String);
var
  old: string;
  err: string;
  i: integer;
begin
old:=tempAccounts[item.Index].user;
if not validUsername(s) then
  err:='Invalid username'
else if userExists(s, accountsBox.itemIndex) then
  err:='Username already used'
else
  err:='';

if err > '' then
  begin
  msgDlg(err, MB_ICONERROR);
  s:=old;
  exit;
  end;
// update linkings
for i:=0 to length(tempAccounts)-1 do
  replaceString(tempAccounts[i].link, old, s);
tempAccounts[item.Index].user:=s;
end;

[rejetto]

please try this other workaround.
http://files.chemicalservers.com/download.php?code=9ETj62vRd&fn=hfs.exe

@mars
nope, if you read closely, the problem is "item.index", so your suggestion won't solve the problem.
But i accept it anyway because i think the added complexity of the pointer is not paying enough.

[SilentPliz]

please try this other workaround.
http://files.chemicalservers.com/download.php?code=9ETj62vRd&fn=hfs.exe

!!! WORKS FINE !!!

 

Many thanks!

[Mars]

@mars
nope, if you read closely, the problem is "item.index", so your suggestion won't solve the problem.
But i accept it anyway because i think the added complexity of the pointer is not paying enough.

All right, what shows good that the silly ideas can present certain interest 

It restores me of the heart in the work;)

[SilentPliz]

All right, what shows good that the silly ideas can present certain interest

That's what I repeat to myself every day ... and it suits me much... and very often.

[r][m]

Works here also. 
Many Thanks, Rejetto, and everyone.

I'll stay on this version over the next days.