Uncaught ReferenceError: writeHTML is not defined

Question

Sorry my English, I am of another nationality

I have a code in js but unfortunately it does not work because of this warning How to fix it and make it work, thank you

You're right, excuse me, here's the full code, i included this script in phpBB 3.1

These are the affected files

1) overall_footer_after.html

2) overall_header_page_body_before.html

3) scrollamp.js

1) In overall_footer_after.html and loaded the scrollamp.js file this way

<!-- INCLUDEJS scrollamp.js -->

2) This is the code I put in the file: overall_header_page_body_before.html

<div class="forabg">
    <div class="inner">
        <ul class="topiclist">
            <li class="header"><dl><dt>News</dt></dl></li>
        </ul>
            <div class="postbody" style="width: 100%">              
                <div class="post bg2">
<div class="avviso" style="padding: 0px; width: 400px; height: 200px; margin: 0 auto 0 auto;text-align:center; border: #00ffff 0px solid;">
<script type="text/javascript">
innerHTML()
</script>
</div>
                    </div>
                </div>
            </div>
    </div>

3) And this is the code I put in the file scrollamp.js

function innerHTML() {
document.write("<marquee width\"400\" height\"200\" direction=\"Up\" scrollamount=\"1\" scrolldelay=\"1\">");
document.write("<p style=\"font-size:25px;font-weight: bold;color:red;text-align:center;text-decoration:underline blink;\">.:| AVVISO IMPORTANTE |:.<\/p>");
// document.write("<br\/>");
document.write("<p style=\"font-size:15px;font-weight: bold;color:#000;text-align:center;\">Tutti gli Utenti Registrati, dovranno effettuare almeno 5 messaggi per visualizzare tutto il forum<\/p>");
document.write("<\/marquee>");
}

We cannot see your .js file loaded anywhere. It has to be loaded before the call to the writeHTML. Also you can use innerHTML or at least only have ONE document.write — mplungjan, Jun 03 '17 at 16:26
@mplungjan https://developer.mozilla.org/en-US/docs/Glossary/Hoisting — j08691, Jun 03 '17 at 16:27
@j08691 https://jsfiddle.net/mplungjan/vdqd1L8c/ vs https://jsfiddle.net/mplungjan/73h5y45v/ — mplungjan, Jun 03 '17 at 17:14
If you are using INCLUDEJS - then please tag or link the framework or similar using that syntax — mplungjan, Jun 03 '17 at 17:16

score 0 · Answer 1 · answered Jun 03 '17 at 16:56

I am trying to be nice so I will answer your question. However, PLEASE DON'T EXPECT PEOPLE TO ANSWER SUCH AN AMBIGUOUS QUESTION.
You have not provided your full html code so no one can give any actual suggestion to you.

Here is what I suspect your problem might be:

Your javascript file is fine but you need to import it before using it.

ReferenceError: writeHTML is not defined

This kind of error will only be thrown when, literally, the object has not yet been defined i.e. you have not imported it yet.

To import a script do this:

<script src="fileName.js"></script>

Thats all I could tell from the informations you have provided.

@giacobbecovatta Make sure your scrollamp.js file is located at style/template/ — ed9w2in6, Jun 03 '17 at 23:09

Uncaught ReferenceError: writeHTML is not defined

1 Answers1