Skip to content

Commit

Permalink
Fixed issue #7643: Answer code is not set in comment field headings w…
Browse files Browse the repository at this point in the history 
…hen exporting responses
  • Loading branch information
@c-schmitz
c-schmitz committed Mar 12, 2013
1 parent 5a28cad commit 8d01ad2
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions application/helpers/admin/exportresults_helper.php
View file
Expand Up @@ -1005,16 +1005,10 @@ public function getCodeFieldSubHeading(SurveyObj $survey, FormattingOptions $oOp
{
$subHeading .= ' '.$this->getOtherSubHeading();
}
elseif (!$isComment)
else
{
$subHeading .= ' ['.$answerCode.']';
}
if (isset($isComment) && $isComment == true)
{
$subHeading .= ' '.$this->getCommentSubHeading();
$comment = false;
}

break;

case ':':
Expand Down

4 comments on commit 8d01ad2

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2.1 can use viewHelper::getFieldCode.

Maybe move viewHelper in 2.05 (with getImageLink function too) ?

Oups ... Question calss used, the can't use existing function ;)

@SamMousa
Copy link
Contributor

@SamMousa SamMousa commented on 8d01ad2 Mar 15, 2013 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK,
Then maybe it's best to start using only one function in 2.05 ?
Or if we need to remork whole, waiting for 2.1 ?

@SamMousa
Copy link
Contributor

@SamMousa SamMousa commented on 8d01ad2 Mar 15, 2013 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.