[pywin32-bugs] [ pywin32-Bugs-1648655 ] Wrong args order with event handler

[SourceForge.net <no...@so...>]

Bugs item #1648655, was opened at 2007-01-31 21:09
Message generated for change (Comment added) made by mhammond
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=551954&aid=1648655&group_id=78018

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: com
Group: None
>Status: Pending
Resolution: None
Priority: 5
Private: No
Submitted By: atsuo ishimoto (ishimoto)
Assigned to: Nobody/Anonymous (nobody)
Summary: Wrong args order with event handler

Initial Comment:
Order of args doesn't match with prototype generated by makepy.

To reproduce, please run script below and Save workbook on the Excel.

# <<<<<< BEGIN >>>>>>
import win32com.client

class WorkbookEvent:
    def OnBeforeSave(self, SaveAsUI, Cancel):
        print "OnBeforeSave", SaveAsUI, Cancel
        return False

xl = win32com.client.Dispatch("Excel.Application")
xl.Visible = 1
wb = DispatchWithEvents(xl.Workbooks.Add(), WorkbookEvent)
while 1:
    pythoncom.PumpWaitingMessages()

# <<<<<< END >>>>>>

When I save workbook, WorkbookEvent.OnBeforeSave called with 
SaveAsUI=False and Cancel=False. But on SaveAs, args passed are 
SaveAsUI=False and Cancel=True, which should be SaveAsUI=True and 
Cancel=False. 

Python2.4.3,pywin32-210,MS-Excel 2000



----------------------------------------------------------------------

>Comment By: Mark Hammond (mhammond)
Date: 2007-07-28 17:17

Message:
Logged In: YES 
user_id=14198
Originator: NO

Sorry, but I can't see a bug of ours here.  It sounds like the type info
Excel gives out is wrong - otherwise I'd expect *all* params to be reversed
for all events, and I don't think that is the case.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=551954&aid=1648655&group_id=78018