Strip slashes from question title numbers
Merge request reports
Activity
requested review from @marten
@roqua/Reviewers Mocht de regex een geval overslaan merken we dat bij de questionnaire repo reviews.
mentioned in commit 3e586df0
286 287 end 287 288 288 289 def strip_p_tag(text) 289 text.gsub(/^<p>(.*)<\/p>\n?$/, "\\1") 290 text.gsub /^<p>(.*)<\/p>\n?$/, '\1' 291 end 292 293 def strip_question_number_slashes(quests) 294 quests.transform_values! do |value| 295 value.gsub /^(\s*\d+)\\/, '\1' ik zou dit zelf in generate_question_titles erbij zetten, gezien het bij het strippen van de html hoort, oftwel get converteren van markdown naar text.
Maar is me zo niet meteen duidelijk waarom hier nog titles worden gebruikt, dacht dat in de seeds alleen maar cft's werden gebruikt.
iig moeten die cft's ook allemaal gestript lijkt me. de functie zou dus markdown_to_text moeten heten en op meer plekken aangeroepen moeten worden. option descriptions?
edit: waarbij d\us de eerste regel niet meer geld, generate_question_titles zou markdown_to_text moeten aanroepen.
Edited by Henk van der Veen
