View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
08377Bug reportsImport/Exportpublic2013-12-09 15:23
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.05+ 
Target Version2.05+Fixed in Version2.05+ 
Summary08377: Invalid XML file if debugSQL is set
Description

When you export lss file with debug SQL, an invalid lss file is produced. This break import of xml file ig debug>1.

Steps To Reproduce

Set your debug to more than 2 and trye to import
http://bugs.limesurvey.org/file_download.php?file_id=5870&type=bug
(from : http://bugs.limesurvey.org/view.php?id=8324)

If you look at end of lss file , you see

[GOOD XML FILE]
</document>
<!-- start log messages -->
<table class="yiiLog" ......
[...]
/]]>/
</script>

Additional Information

Here its' for 2.05, think it's same for 2.00

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)131116
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionMysql 5
Server OS (if known)debian/linux (for importing)
Webserver software & version (if known)apache
PHP VersionPHP Version 5.3

Users monitoring this issue

There are no users monitoring this issue.

Activities

mdekker

mdekker

2013-11-19 12:33

reporter   ~27224

See controllers/admin/export.php line 301, this is also in remotecontrol. It should probably be moved to a different place so all actions that need this can access it.

DenisChenu

DenisChenu

2013-11-19 13:09

developer   ~27227

Alternative (if work): setconfig(debug) to 0 ? when needed :)

c_schmitz

c_schmitz

2013-11-21 11:53

administrator   ~27287

I like the solution by mdekker better.
mdekker, can you take care of it?

DenisChenu

DenisChenu

2013-11-21 18:06

developer   ~27298

Maybe in an helper ?
deactivateDebugOutput()

But which helper ? viewHelper.php ?

DenisChenu

DenisChenu

2013-11-21 18:07

developer   ~27299

mdekker : if you don't have time : assign it to me, nope.

c_schmitz

c_schmitz

2013-11-27 16:15

administrator   ~27435

Apparently mdekker is busy currently. Denis, please take it.

DenisChenu

DenisChenu

2013-11-27 16:21

developer   ~27436

OK,

Add this to an helper but same than mdekker if possible. Elese copy/paste mdekker code :)

DenisChenu

DenisChenu

2013-11-29 18:13

developer   ~27457

Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&amp;id=13529

mdekker

mdekker

2013-12-09 09:15

reporter   ~27524

Thanks for taking care of this :)

DenisChenu

DenisChenu

2013-12-09 09:15

developer   ~27525

nope ;)
It's your code here.

c_schmitz

c_schmitz

2013-12-09 15:23

administrator   ~27527

2.05 Build 131209 released.

Related Changesets

LimeSurvey: 2.05 6608fba7

2013-11-29 17:13:46

DenisChenu

Details Diff
Fixed issue 08377: Invalid XML file if debugSQL is set (Thanks to mdekker)
Dev: added viewHelper::disableHtmlLogging();
Affected Issues
08377
mod - application/controllers/admin/export.php Diff File
mod - application/helpers/viewHelper.php Diff File

Issue History

Date Modified Username Field Change
2013-11-16 09:31 DenisChenu New Issue
2013-11-19 12:33 mdekker Note Added: 27224
2013-11-19 13:09 DenisChenu Note Added: 27227
2013-11-21 11:53 c_schmitz Note Added: 27287
2013-11-21 11:53 c_schmitz Assigned To => mdekker
2013-11-21 11:53 c_schmitz Status new => feedback
2013-11-21 18:06 DenisChenu Note Added: 27298
2013-11-21 18:06 DenisChenu Status feedback => assigned
2013-11-21 18:07 DenisChenu Note Added: 27299
2013-11-21 18:07 DenisChenu Status assigned => feedback
2013-11-22 11:29 c_schmitz Target Version => 2.05+
2013-11-27 16:15 c_schmitz Assigned To mdekker => DenisChenu
2013-11-27 16:15 c_schmitz Status feedback => assigned
2013-11-27 16:15 c_schmitz Note Added: 27435
2013-11-27 16:21 DenisChenu Note Added: 27436
2013-11-29 18:13 DenisChenu Changeset attached => LimeSurvey 2.05 6608fba7
2013-11-29 18:13 DenisChenu Note Added: 27457
2013-11-29 18:13 DenisChenu Resolution open => fixed
2013-11-29 18:14 DenisChenu Status assigned => resolved
2013-11-29 18:14 DenisChenu Fixed in Version => 2.05+
2013-12-09 09:15 mdekker Note Added: 27524
2013-12-09 09:15 DenisChenu Note Added: 27525
2013-12-09 15:23 c_schmitz Note Added: 27527
2013-12-09 15:23 c_schmitz Status resolved => closed