View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
07950 | Bug reports | Import/Export | public | 2013-06-24 21:55 | 2013-09-24 14:20 |
Reporter | mfaber | Assigned To | adamzammit | ||
Priority | normal | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.05 RC | ||||
Fixed in Version | 2.05 RC | ||||
Summary | 07950: queXML export produces unusable PDF, "/**/" on every line, 2.00+ seems ok | ||||
Description | When exporting a Survey using quexml-export, the resulting PDF contains lots of This only seems to be a problem in beta 2.05. | ||||
Steps To Reproduce | choose any survey | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 10 | ||||
Complete LimeSurvey version number (& build) | 130624 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Chrome | ||||
Database type & version | MySQL 5.0.10 | ||||
Server OS (if known) | Win7 | ||||
Webserver software & version (if known) | Apache/2.4.3 (Win32) | ||||
PHP Version | PhP 5.4.7 | ||||
OK, I think I found the problem.... in quexmlpdf.php the following line pulls the style information from the style-file. Unfortunately, in LS 2.05 there are some css/JS files registered using e.g. registerCssFile() in CClientscript.php. Now, together with the style information from the style-file comes alist of these registered files and some other stuff created by the renderer. The whole style variable looks like this on my system: <link rel="stylesheet" type="text/css" href="http://localhost/github/limesurvey/tmp/assets/b52d3e57/css/bootstrap.min.css" /> Only the middle part is the style information from the file and the last bit of these lines is where the /**/ is coming from in the quexml-PDF. I am unsure how to deal with this...Adam, can you please advice? Do we need to pull the styles through the renderer? Apply filtering before passing it to tcpdf..? |
|
Hi mfaber, I see this problem too. I'll post a solution shortly. Regards, |
|
Hi again mfaber, Attached is a new quexmlpdf.php file that doesn't use the renderer. I think this is more appropriate as the class style is specific to queXMLPDF and doesn't need to be separated. Let me know if this works and I'll update 2.05. Regards, |
|
Works! Yeeha :) Thanks, |
|
Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12769 |
|
Suggest to also remove the now obsolete quexmlpdf_view.php from the repository... |
|
Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12770 |
|
2.05RC1 released |
|
LimeSurvey: 2.05 ad758f0e 2013-07-23 08:17 Details Diff |
Fixed issue 07950: queXML export produces unusable PDF, "/**/" on every line, 2.00+ seems ok |
Affected Issues 07950 |
|
mod - application/libraries/admin/quexmlpdf.php | Diff File | ||
LimeSurvey: 2.05 bc1c6135 2013-07-23 08:42 Details Diff |
Fixed issue 07950: queXML export produces unusable PDF, "/**/" on every line, 2.00+ seems ok Dev Remove now obsolete view |
Affected Issues 07950 |
|
rm - application/views/admin/export/quexmlpdf_view.php | Diff |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-06-24 21:55 | mfaber | New Issue | |
2013-07-18 13:12 | mfaber | File Added: quexmlpdf_784912_en.pdf | |
2013-07-19 14:18 | mfaber | Note Added: 25815 | |
2013-07-19 14:18 | mfaber | Assigned To | => adamzammit |
2013-07-19 14:18 | mfaber | Status | new => assigned |
2013-07-19 14:18 | mfaber | Issue Monitored: mfaber | |
2013-07-23 06:54 | adamzammit | Note Added: 25846 | |
2013-07-23 06:54 | adamzammit | Status | assigned => acknowledged |
2013-07-23 07:13 | adamzammit | File Added: quexmlpdf.php | |
2013-07-23 07:15 | adamzammit | Note Added: 25847 | |
2013-07-23 07:17 | adamzammit | Issue Monitored: adamzammit | |
2013-07-23 08:11 | mfaber | Note Added: 25848 | |
2013-07-23 08:17 | adamzammit | Changeset attached | => LimeSurvey 2.05 ad758f0e |
2013-07-23 08:17 | adamzammit | Note Added: 25849 | |
2013-07-23 08:17 | adamzammit | Resolution | open => fixed |
2013-07-23 08:30 | mfaber | Note Added: 25850 | |
2013-07-23 08:42 | adamzammit | Changeset attached | => LimeSurvey 2.05 bc1c6135 |
2013-07-23 08:42 | adamzammit | Note Added: 25851 | |
2013-07-23 13:43 | mfaber | Status | acknowledged => resolved |
2013-09-23 13:37 | c_schmitz | Fixed in Version | => 2.05 RC |
2013-09-24 14:20 | c_schmitz | Note Added: 26340 | |
2013-09-24 14:20 | c_schmitz | Status | resolved => closed |
2021-08-02 21:00 | guest | Bug heat | 6 => 10 |