CoreCommunity ExtensionsIncubatorDistributionsTYPO3 4.5 ProjectsTYPO3 4.7 ProjectsTYPO3 6.0 ProjectsTYPO3 6.1 ProjectsTYPO3 6.2 Projects (+)

Bug #12866

database records include files

Added by iggy over 2 years ago. Updated over 2 years ago.

Status:Accepted Start date:2011-02-08
Priority:Must have Due date:
Assignee:Stefan Froemken % Done:

0%

Category:- Spent time: -
Target version:-
Votes: 0

Description

when trying to include external scripts i found that the paths chosen with the wizzard render unusable as there's a need for absolute filesystem paths to include the files...

fixed that in class.tx_sfjquery_p1.php line 388 ff.:

if(is_file(trim(PATH_site.$row['domscriptfile']))) $this->domscript .= file_get_contents(trim(PATH_site.$row['domscriptfile']))."\n";
$this->outscript = $row['outscript'].CHR;
if(is_file(trim(PATH_site.$row['outfile']))) $this->outscript .= file_get_contents(trim(PATH_site.$row['outfile']))."\n";
$this->content = $row['content'].CHR;
if(is_file(trim(PATH_site.$row['htmlfile']))) $this->content .= file_get_contents(trim(PATH_site.$row['htmlfile']))."\n";
$this->ts = $row['ts'].CHR;
if(is_file(trim(PATH_site.$row['tsfile']))) $this->ts .= file_get_contents(trim(PATH_site.$row['tsfile']))."\n";
$this->style = $row['css'].CHR;
if(is_file(trim(PATH_site.$row['cssuifile']))) $this->conf['cssFile'] = trim($row['cssuifile']);

there also was a typo in line 388 where $row['domscriptfile'] was named $row['domfile'] obviously by mistake.

would be nice if this little change would be incorporated within the next release :)

Associated revisions

Revision 43751
Added by Stefan Froemken over 2 years ago

Bug: #12866: Wrong variable: domscriptfile instead of domscript

Revision 43751
Added by Stefan Froemken over 2 years ago

Bug: #12866: Wrong variable: domscriptfile instead of domscript

Revision 43758
Added by Stefan Froemken over 2 years ago

- Bug #12866: domfile instead of domscriptfile
- updated to jQuery 1.4.4
- updated to jquery-ui 1.8.9
- updated all jquery samples to new jquery version
- suppress_icons = 1 for hiding images below selectbox
- imagegeneration has its own class now, because it has nothing to do with jquery
- images will be generated with cObj now
- changed default image directory to uploads/media/ in flexform

Revision 43758
Added by Stefan Froemken over 2 years ago

- Bug #12866: domfile instead of domscriptfile
- updated to jQuery 1.4.4
- updated to jquery-ui 1.8.9
- updated all jquery samples to new jquery version
- suppress_icons = 1 for hiding images below selectbox
- imagegeneration has its own class now, because it has nothing to do with jquery
- images will be generated with cObj now
- changed default image directory to uploads/media/ in flexform

History

Updated by Stefan Froemken over 2 years ago

  • Status changed from New to Accepted

What do you meen with external scripts? Something like http://[other webpage]/script/jquery.js?
Thank you for bug reporting regarding domscriptfile. I have fixed it in rev. 43751.

Also available in: Atom PDF