Opened 13 years ago

Last modified 12 years ago

#2199 closed Bug

_ExcelSheetAddNew() abends if specified sheet already exists — at Version 4

Reported by: Spiff59 Owned by:
Milestone: 3.3.9.5 Component: Standard UDFs
Version: 3.3.8.1 Severity: None
Keywords: Cc:

Description (last modified by water)

As the Excel UDF is being brushed up this seems to be the right moment to reopen and solve this ticket.

Change History (4)

comment:1 Changed 13 years ago by anonymous

I suppose I could have made the @error = 2 test a single-line If/Then statement, if anyone cares...

comment:2 Changed 13 years ago by trancexx

  • Resolution set to Rejected
  • Status changed from new to closed

That function doesn't follow its specification regarding return values.
If you want to submit some code then either do it correctly or don't do it at all. I understand the desire to be useful but please don't do it by adding new bugs.

I'll close this as "Rejected", and I'm hoping you will report it again, only correctly next time. By correctly I mean with explanation of the bug, reproducer (where is it now?), and suggestions if you have any.

comment:3 Changed 13 years ago by anonymous

Oh, the SetError() returns... shame on me.
Leaving it as rejected works for me.

comment:4 Changed 12 years ago by water

  • Component changed from AutoIt to Standard UDFs
  • Description modified (diff)
  • Resolution Rejected deleted
  • Status changed from closed to reopened
Note: See TracTickets for help on using tickets.