jethro-pmm icon indicating copy to clipboard operation
jethro-pmm copied to clipboard

Undefined index heading_text when saving run sheet

Open tbar0970 opened this issue 7 years ago • 3 comments

111

Undefined index: heading_text Line 58 of /home/jethro/code/2.18.3-RC4/views/view_8_services.class.php

REFERER: https://xyz.easyjethro.com.au/?view=services&date=2017-03-12&congregationid=6&editing=1 USER_AGENT: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/602.4.8 (KHTML, like Gecko) Version/10.0.3 Safari/602.4.8

REQUEST: Array ( [view] => services [date] => 2017-03-12 [congregationid] => 6 [editing] => 1 [save_service] => 1 [title] => Array ( ... )

[componentid] => Array ( ... )

[length_mins] => Array ( ... )

[show_in_handout] => Array ( ... )

[note] => Array ( ... )

[personnel] => Array ( ... )

[service_comments] => )

BACKTRACE: Array ( [0] => Array ( [file] => /home/jethro/code/2.18.3-RC4/include/system_controller.class.php [line] => 127 [function] => processView [class] => View_services [type] => -> [args] => Array ( )

   )

[1] => Array ( [file] => /home/jethro/code/2.18.3-RC4/index.php [line] => 58 [function] => run [class] => System_Controller [type] => -> [args] => Array ( )

   )

)

tbar0970 avatar Mar 09 '17 01:03 tbar0970

+1 - safari again

tbar0970 avatar May 06 '17 07:05 tbar0970

I got this just now, after I edited a run sheet and the only change I made was editing the service comments... could that be it?

tbar0970 avatar Jul 26 '18 06:07 tbar0970

Doesnt seem to be

tbar0970 avatar Aug 04 '20 04:08 tbar0970

I experienced an issue like this one today, in the context of a bug that was preventing JethroServicePlanner.onsubmit to run before the form submitted. The heading_text elements are added in to the request by this function. So I suspect the original errors are some similar situation, where the form is 'natively' submitting without the onSubmit function running.

tbar0970 avatar Jun 20 '23 12:06 tbar0970

Despite not having tracked down the original trigger, I'm fairly confident this change will make this one go away.

tbar0970 avatar Jun 20 '23 12:06 tbar0970