Hook Request

This forum is for programmers who have questions about the source code.
Post Reply
User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Hook Request

Post by drtech » Thu Mar 21, 2013 7:25 pm

Please add

Plugins.HookAddCode(this, "FormStatementOptions_Load_end");

to FormStatmentOptions.cs | Load | Line 844

Thanks
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Hook Request

Post by drtech » Mon Mar 25, 2013 4:18 pm

If we could get this added soon I would appreciate it! Thanks!
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Hook Request

Post by drtech » Tue Mar 26, 2013 7:37 pm

I need a second Hook also please

if (Plugins.HookMethod(this, "FormOpenDental.FillPatientButton_Popups", pat, PopupEventList, patChanged))
{
return;
}


in FormOpendental.cs, around line 2304, right above "if (!patChanged)"

Thanks
Last edited by drtech on Wed Mar 27, 2013 7:21 pm, edited 1 time in total.
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

michael
Posts: 38
Joined: Wed Aug 04, 2010 8:49 am

Re: Hook Request

Post by michael » Wed Mar 27, 2013 4:17 pm

These hooks have been added.

Ln 839 of FormStatementOptions.cs

Code: Select all

Plugins.HookAddCode(this,"FormStatementOptions_Load_end");
Ln 2306 of FormOpenDental.cs

Code: Select all

if(Plugins.HookMethod(this,"FormOpenDental.FillPatientButton_popups",pat,PopupEventList)) {
	return;
}
They will be in 13.1.18.

We have also updated our hook page (Plug-in Patterns) http://opendental.com/manual/patternplugins.html

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Hook Request

Post by drtech » Wed Mar 27, 2013 7:20 pm

Great...one change though on the second hook that I updated earlier, probably when you were in the process of adding it and so missed my change...


please pass patChanged also in the hook and move the hook up a couple of lines, above the "if (!patChanged)" evaluation instead of just below it around line 2304, Thanks

if (Plugins.HookMethod(this, "FormOpenDental.FillPatientButton_popups", pat, PopupEventList, patChanged))
{
return;
}


Thanks for updating the plugin page manual! I will be using a goto hook also like Jordan and I discussed and you posted about in the manual...let me post it in another thread.
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Hook Request

Post by drtech » Wed Apr 03, 2013 8:15 pm

if this change could be made I would really appreciate it so I can release my plugin for testing to others...thanks.
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Hook Request

Post by drtech » Mon Apr 08, 2013 5:42 am

as stated above a week or so ago...


please pass patChanged also in the hook and move the hook up a couple of lines, above the "if (!patChanged)" evaluation instead of just below it around line 2304, Thanks

if (Plugins.HookMethod(this, "FormOpenDental.FillPatientButton_popups", pat, PopupEventList, patChanged))
{
return;
}
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

michael
Posts: 38
Joined: Wed Aug 04, 2010 8:49 am

Re: Hook Request

Post by michael » Mon Apr 08, 2013 9:40 am

It'll be in 13.1.19.

Ln 2303 of FormOpenDental.cs

Code: Select all

			if(Plugins.HookMethod(this,"FormOpenDental.FillPatientButton_popups",pat,PopupEventList,patChanged)) {
				return;
			}

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Hook Request

Post by drtech » Mon Apr 08, 2013 11:32 am

thanks!
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

Post Reply