Nemcio/Grav
1
0
Fork 0
mirror of https://github.com/getgrav/grav.git synced 2026-03-05 03:51:50 +01:00
Code Issues Packages Projects Releases Activity

Specify type in inlineJS (#1683)

Browse Source 
This commit allows users to specify type to inlineJS via a 4th parameter . 

This might be useful for those who would like to output encoded json microdata which require to use 
<script type="application/ld+json"> to be valid. 

Exemple usage: $assets->addInlineJs($outputjson, 100,'', "application/ld+json");
which will output: <script type="application/ld+json">
This commit is contained in:
Paul Massendari
2017-10-11 19:05:01 +02:00
committed by Andy Miller
parent 7ccba91792
commit df63a85f3e
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
system/src/Grav/Common/Assets.php
View File

@@ -470,7 +470,7 @@ class Assets
*
* @return $this
*/
public function addInlineJs($asset, $priority = null, $group = null)
public function addInlineJs($asset, $priority = null, $group = null, $attributes = null)
{
$asset = trim($asset);
@@ -485,7 +485,8 @@ class Assets
'asset' => $asset,
'priority' => intval($priority ?: 10),
'order' => count($this->js),
'group' => $group ?: 'head'
'group' => $group ?: 'head',
'type' => $attributes ?: '',
];
// check for dynamic array and merge with defaults
@@ -667,7 +668,10 @@ class Assets
}
if ($inline_js) {
$output .= "\n<script>\n" . $inline_js . "\n</script>\n";
if ($inline['type']){
$attributeString = " type=\"" . $inline['type'] . "\"";
}
$output .= "\n<script" . $attributeString . ">" . $inline_js . "\n</script>\n";
}
return $output;