Project

General

Profile

Bug #7134

When api_get_setting('include_asciimathml_script') == 'true') the page is broken.

Added by Julio Montoya over 7 years ago. Updated over 7 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Styles, Layout & Accessibility
Target version:
Start date:
06/05/2014
Due date:
% Done:

100%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

<script type="text/javascript">var translateOnLoad = false ;</script>
<!DOCTYPE html>
<!--[if lt IE 7]> <html lang="en" class="no-js lt-ie9 lt-ie8 lt-ie7"> <![endif]-->
<!--[if IE 7]>    <html lang="en" class="no-js lt-ie9 lt-ie8"> <![endif]-->
<!--[if IE 8]>    <html lang="en" class="no-js lt-ie9"> <![endif]-->
<!--[if gt IE 8]><!--><html lang="en" class="no-js"> <!--<![endif]-->
<head>

The "<script type="text/javascript">var translateOnLoad = false ;</script>" code is printed.

Tested in main/messages/new_message.php

Associated revisions

Revision 26ca82fa (diff)
Added by Julio Montoya over 7 years ago

Fixing include_asciimathml_script setting see #7134

History

#1

Updated by Julio Montoya over 7 years ago

  • Status changed from New to Needs testing
  • % Done changed from 0 to 50
#2

Updated by Yannick Warnier over 7 years ago

I'm not able to reproduce (before pulling your code), so I'm not sure how to verify it. For me, it didn't really exist (at least in the latest 1.9.8). I tried the following pages:
  • homepage
  • course homepage
  • portal new edition page (with FCKeditor)
  • document edition and list
  • agenda
  • social network

None of these pages had the erroneous header... !?

#3

Updated by Julio Montoya over 7 years ago

Yannick Warnier wrote:

I'm not able to reproduce (before pulling your code), so I'm not sure how to verify it. For me, it didn't really exist (at least in the latest 1.9.8). I tried the following pages:
  • homepage
  • course homepage
  • portal new edition page (with FCKeditor)
  • document edition and list
  • agenda
  • social network

None of these pages had the erroneous header... !?

I put in the description the URL: main/messages/new_message.php

#4

Updated by Yannick Warnier over 7 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee set to Julio Montoya
  • % Done changed from 50 to 100

Yes, found it from the private report. The issue was on redaction of a new internal message from the social network.
Your patch fixes it, so great. Approving and closing.

Also available in: Atom PDF