Skip to content

Commit a6e1adf

Browse files
committedJan 18, 2012
Fixes to Survey tests and merged code from 7.10
1 parent 1ab73e2 commit a6e1adf

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed
 

‎docs/migration.txt

+5
Original file line numberDiff line numberDiff line change
@@ -422,3 +422,8 @@ getEditForm now returns a WebGUI::FormBuilder object
422422
Show Performance Indicators
423423
==========================
424424
This setting is removed, as the Plack debug console shows this for us.
425+
426+
WebGUI::Asset::Wobject::Survey
427+
==========================
428+
The surveyJSON method conflicted with the new Moose accessor. In WebGUI 8,
429+
the old surveyJSON is called getSurveyJSON.

‎lib/WebGUI/Asset/Wobject/Survey.pm

+2-2
Original file line numberDiff line numberDiff line change
@@ -2439,8 +2439,8 @@ Extend the base method to include custom question types added to this Survey.
24392439
sub exportAssetData {
24402440
my $self = shift;
24412441
my $asset_data = $self->SUPER::exportAssetData();
2442-
my $questions = $self->surveyJSON->questions();
2443-
my $multiple_choice = $self->surveyJSON->multipleChoiceTypes();
2442+
my $questions = $self->getSurveyJSON->questions();
2443+
my $multiple_choice = $self->getSurveyJSON->multipleChoiceTypes();
24442444
my %question_types = ();
24452445
my $get_question = $self->session->db->prepare('select answers from Survey_questionTypes where questionType=?');
24462446
foreach my $question (@{ $questions }) {

‎t/Asset/Wobject/Survey/package.t

+2-2
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ my $import_node = WebGUI::Asset->getImportNode($session);
2727
my $survey = $import_node->addChild( { className => 'WebGUI::Asset::Wobject::Survey', } );
2828
WebGUI::Test->addToCleanup($survey);
2929

30-
my $sJSON = $survey->surveyJSON;
30+
my $sJSON = $survey->getSurveyJSON;
3131

3232
# Load bare-bones survey, containing a single section (S0)
3333
$sJSON->update([0], { variable => 'S0' });
@@ -57,7 +57,7 @@ $asset_data->{question_types}->{fingers} = clone $asset_data->{question_types}->
5757
$survey->importAssetCollateralData($asset_data);
5858

5959
$survey = $survey->cloneFromDb;
60-
my $multipleChoiceTypes = $survey->surveyJSON->multipleChoiceTypes;
60+
my $multipleChoiceTypes = $survey->getSurveyJSON->multipleChoiceTypes;
6161

6262
ok exists $multipleChoiceTypes->{fingers}, 'fingers type imported as package collateral data';
6363
ok exists $multipleChoiceTypes->{toes}, 'still have toes, too';

0 commit comments

Comments
 (0)
Please sign in to comment.