Modify

Opened 17 years ago

Closed 17 years ago

#645 closed Feature Request (Completed)

IniDelete 3rd param doesn't accept Default keyword

Reported by: Rob Saunders <therks@…> Owned by: Valik
Milestone: 3.2.13.10 Component: AutoIt
Version: Severity: None
Keywords: inidelete default Cc:

Description

I was trying to write a wrapper for IniDelete and found I had to resort to an if statement, and two separate calls to IniDelete to allow it to delete whole sections.
Not sure if this is a bug or not, but it's unexpected behaviour on my part at least. A Simple repro of the "bug":

IniWrite(@tempdir & '\temp.ini', 'hello', 'world', 'foobar')
IniDelete(@tempdir & '\temp.ini', 'hello', Default)
ConsoleWrite(IniRead(@tempdir & '\temp.ini', 'hello', 'world', ''))

Console should be empty but instead we get "foobar" back.

Bug or feature request I'm not sure.

Attachments (0)

Change History (7)

comment:1 by J-Paul Mesnage, 17 years ago

Resolution: No Bug
Status: newclosed

as stated in "Default keyword" doc

When used as a parameter passing the behavior is specify in the corresponding AutoIt doc function.

so Default is not in use in this function so it converted to "Default" so the IniDelete is just erasing something which does not exist and return 1 as after completion the .ini does not anymore such key !!!

Perhaps after Valik rewrite this can change.

comment:2 by J-Paul Mesnage, 17 years ago

as stated in "Default keyword" doc

When used as a parameter passing the behavior is specify in the corresponding AutoIt doc function

sorry for these bad display

in reply to:  1 ; comment:3 by Valik, 17 years ago

Resolution: No Bug
Status: closedreopened

Replying to Jpm:

Perhaps after Valik rewrite this can change.

Or I could just change it now since supporting the Default keyword has nothing to do with the INI implementation.

comment:4 by Valik, 17 years ago

Owner: set to Valik
Status: reopenedassigned

in reply to:  3 comment:5 by J-Paul Mesnage, 17 years ago

Replying to Valik:

Replying to Jpm:

Perhaps after Valik rewrite this can change.

Or I could just change it now since supporting the Default keyword has nothing to do with the INI implementation.

so it is an improvement not a bug the closing of the bug was I think the rigth thing before deciding to improve as you suggest

comment:6 by Valik, 17 years ago

Type: BugFeature Request
Version: 3.2.13.9

There. It's a feature request now.

comment:7 by Valik, 17 years ago

Milestone: 3.2.13.10
Resolution: Completed
Status: assignedclosed

Added in version: 3.2.13.10

Modify Ticket

Action
as closed The owner will remain Valik.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.