Hi
Ok, I've got ls sync installed and working. I can create a survey instance, execute the survey, and see the results in my drupal site. Really cool!

After doing this, and answering the survey once, I tried to edit the content type into which the survey results are stored. When I try to save this content type's settings, I get a WSOD, and looking in error_log I see
Call to undefined function ls_survey_answ_options_validate() in /home4/xxx/public_html/includes/form.inc on line 1465

Help?

The survey still works. I just can't change the features of its content type. I wanted to modify the location settings of the content type so they would not show when you view the survey results node.

The settings in my content type are:
-- open into an iFrame
-- Automatically overwrite
-- automatically fit to the user window
-- expiration period: 1 day
-- this specific page:
-- response title options: all defaults
-- response display page options: all defaults
-- response sync options:
-- -- specific user has been set
-- -- fill the token table is checked. Other checkboxes unchecked.

Thanks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hanksterr7’s picture

Priority: Major » Normal
hanksterr7’s picture

Ok, I figured out what's going on. I changed the priority from Major to Normal as a consequence.

I was trying to edit the content type from admin/structure/types. That is not valid for these content types created from the Survey instance.

Instead, if I open the survey node and click its options tab, that opens up the same form as I see if I were editing the content type from admin/structure/types. But this time, when editing from the survey's options tab, I can save my changes.

Not sure what can be done to disable interacting with the content type from admin/structure/types (or at least pop up a message). WSOD's are not nice.

Thanks

thedut’s picture

Assigned: Unassigned » thedut
Status: Active » Needs review
FileSize
579 bytes
9.14 KB

Hello hanksterr7,

Thank you for reporting this bug.
User should be able to edit the answer content type using either admin/structure/types page or the Survey option tab.
I'm not able to reproduce this issue but thanks to the informations you have provided, I guess this patch may fix it.
Could you confirm ?

hanksterr7’s picture

Hi thedut,

Sorry for the delay in trying out your patch.

I replaced ls_survey.module with the file in the second attachment

I then tried to open an existing survey result content type via http://example.com/admin/structure/types/manage/test-test-type

I get a WSOD, and looking in error_log I see
7-Dec-2013 18:14:08 America/New_York] PHP Fatal error: Call to undefined function limesurvey_sync_disable_overlay() in /home4/example/public_html/sites/all/modules/limesurvey_sync/ls_survey/ls_survey.module on line 533

Prior to making this change, I could open http://example.com/admin/structure/types/manage/test-test-type
but when I try to save, I get a wsod and in error_log I see
7-Dec-2013 18:10:14 America/New_York] PHP Fatal error: Call to undefined function ls_survey_answ_options_validate() in /home4/example/public_html/includes/form.inc on line 1465

I can edit the survey result content type via the options tab with no difficulty at
http://example.com/node/#/options

Hope this helps
-- Hank

thedut’s picture

Status: Needs review » Needs work

Yes, thank you Hank for reporting.
I am gointo to prepare a new patch, trying to kill this issue. I will post it here.

thedut’s picture

Hello Hank,

Could you test this new patch please (just unzip and replace the ls_survey/ls_survey.module file) and report if it solves this issue ?

hanksterr7’s picture

Works! No more WSOD editing the survey result content type directly at admin/structure/types. I can also continue to edit the content type via the Options tab for the Survey content type instance.

Thanks!

thedut’s picture

Great !
I will include this fix into the next LimeSurvey Sync release.

thedut’s picture

Status: Needs review » Patch (to be ported)
thedut’s picture

Status: Patch (to be ported) » Closed (fixed)

Fixed into the LimeSurvey Sync 7.x-1.6 release.