Ed,
Regarding your response that the bizobj cancel must fire....
IMHO...
If we are closing the form and we have confirmed that we want to cancel,
then the status of the bizobj's is a "don't care" issue. In the unlikely
event that they are referenced somewhere else, then we should let that other
reference "worry" about issuing the bizobj.cancel()
Do you not agree?
To test, I changed the dForm BaseForm.confirmChanges method as noted below
and it works great!
Starting at line 157(approx)...
if changedBizList:
queryMessage =
self.getConfirmChangesQueryMessage(changedBizList)
response = dabo.ui.areYouSure(queryMessage, parent=self)
if response == None: ## cancel
# Don't let the form close, or requery happen
return False
elif response == True: ## yes
for biz in changedBizList:
self.save(dataSource=biz.DataSource)
elif response == False: ## no
#LAL - Why to we need this?
pass
#for biz in changedBizList:
#if biz.RowCount:
#self.cancel(dataSource=biz.DataSource)
Thanks,
Larry Long
> -----Original Message-----
> From: [email protected]
> [mailto:[email protected]] On Behalf Of Ed Leafe
> Sent: Thursday, May 07, 2009 7:29 PM
> To: Dabo developers' list
> Subject: Re: [dabo-dev] confirmChanges() == No but we scan away
>
> On May 7, 2009, at 6:21 PM, johnf wrote:
>
> > My DataSet is 1500+ records. And I think it is checking
> every record.
>
> First, a 1500 record data set isn't that large. And I'd
> like to know why you "think" this is happening.
>
> > Maybe I don't know all the possible reasons the message
> would appear.
> > I assume it only appeared if the user was attempting to
> close the form
> > and changes to the data had occurred. If that's true why
> worry about
> > indo-ing the changes. If the user said no to the save.
>
> Because the cancel() method must fire. You cannot
> assume that this bizobj in question is not referenced anywhere else.
>
>
> -- Ed Leafe
>
>
>
>
[excessive quoting removed by server]
_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message:
http://leafe.com/archives/byMID/!~!uenerkvcmdkaaqacaaaaaaaaaaaaaaaaabgaaaaaaaaaafa2fnyupuomnfpiynbeqckaaaaqaaaaxosygrtphuaevtssor2zoaeaa...@charter.net
