Bug #21831
closed
TSpagegen::inline2TempFile($this->makeScript(), 'js')
Added by Claus Harup almost 15 years ago.
Updated about 6 years ago.
Description
I was wondering why you guys changed the function TSpagegen::inline2TempFile not to produce the wrapping <link....../> or <script.... />
Now we have to change at manually in ALL extensions including TSpagegen::inline2TempFile
(issue imported from #M12991)
Yes, indeed. I have experienced the same problem.
It's a change that impacts a lot of existing code.
Funny thing is, the comment above the method says:
@return string <script> or <link> tag for the file.
You are right that it seems to have been a backward-compatibility-break. But since it's been around for some time now, I don't think we can revert it either. So I'd propose to "live with it" as it's now and close this ticket.
(But yes Claus, I feel your pain ... happened that I once had to modify an extension for 4.2 back to the old style and now back to the "new" one again.)
- Status changed from Resolved to Closed
Also available in: Atom
PDF