| Summary | Horde::addScriptFile() uses deprecated language attribute |
| Queue | Horde Base |
| Queue Version | HEAD |
| Type | Bug |
| State | Resolved |
| Priority | 2. Medium |
| Owners | |
| Requester | mwarden (at) gmail (dot) com |
| Created | 08/06/2005 (7295 days ago) |
| Due | |
| Updated | 08/07/2005 (7294 days ago) |
| Assigned | |
| Resolved | 08/07/2005 (7294 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | No |
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Horde::addScriptFile() uses deprecated language attribute
Queue ⇒ Horde Base
New Attachment: Horde.php.diff
compliant. The problem I am having at the moment is this line
(framework/Horde/Horde.php#253):
echo '<!--a75c305b1c0a6022--><script language="JavaScript"
type="text/javascript" src="' .
$url . '"></script>';
As you can see, the output has the invalid attribute 'language'. Is
this in the plans to fix, or are you leaving it in there for (very)
backward compatibility? This line is in the addScriptFile() method
which is used all over the place in Horde_Form (and I'm sure
elsewhere). The langauge attribute was deprecated in HTML4.01, if that
helps in the decision.
Patch attached.