Merge pull request #294 from Ducz0r/lm-sci-674-fix

Import protocol steps' rich text styling when importing protocols [SCI-674]
This commit is contained in:
Luka Murn 2016-11-16 16:59:18 +01:00 committed by GitHub
commit aadcdde417
3 changed files with 6 additions and 5 deletions

View file

@ -60,7 +60,7 @@ function importProtocolFromFile(
// Populate values in the template
if (values !== null) {
_.each(values, function(val, key) {
template.find("[data-val='" + key + "']").text(val);
template.find("[data-val='" + key + "']").append(val);
});
}
@ -134,7 +134,7 @@ function importProtocolFromFile(
var stepGuid = node.attr("guid");
var stepPosition = String(Number.parseInt(node.attr("position")) + 1);
var stepName = node.children("name").text();
var stepDescription = node.children("description").text();
var stepDescription = $(node.children("description")).html();
// Generate step element
var stepEl = newPreviewElement(
@ -431,7 +431,7 @@ function importProtocolFromFile(
stepJson.id = stepId;
stepJson.position = $(this).attr("position");
stepJson.name = $(this).children("name").text();
stepJson.description = $(this).children("description").text();
stepJson.description = $(this).children("description").html();
// Iterate through assets
var stepAssetsJson = [];

View file

@ -53,7 +53,8 @@ module ProtocolsImporter
protocol_json['steps'].values.each do |step_json|
step = Step.create!(
name: step_json["name"],
description: step_json["description"],
description: # Sanitize description HTML
ActionController::Base.helpers.sanitize(step_json['description']),
position: step_pos,
completed: false,
user: user,

View file

@ -14,7 +14,7 @@
<br>
<div class="tab-content">
<div class="tab-pane active" role="tabpanel">
<span data-val="description"></span>
<div data-val="description"></div>
<hr>
<div class="row">
<div data-toggle="tables" class="col-xs-12">