#14 ✓resolved
paul (at wingu)

Failed to load deps.js - incorrect path

Reported by paul (at wingu) | August 26th, 2010 @ 03:45 PM | in 0.9 backlog

When I load the index.html file in Safari 5.0.1 and check the developer Console I notice that it has tried and failed to load "deps.js".

There is a file "kemia/deps.js" created when I run ant build-docs but the paths are different. I'm guessing either the ant build.xml should put the deps.js one folder higher, or the javascript should look for it inside the kemia subfolder ( perhaps goog.basePath="kemia" instead of goog.basePath=""?)

I'm not familiar enough with it to know the appropriate fix.

Comments and changes to this ticket

  • paul (at wingu)

    paul (at wingu) August 26th, 2010 @ 03:46 PM

    imported from github issues:
    To reproduce on my Mac running Snow Leopard:

    Richard-Wests-MacBook-Pro:kemia-testing rwest$ git clone http://github.com/rwest/kemia.git
    Cloning into kemia...
    remote: Counting objects: 3908, done.
    remote: Compressing objects: 100% (1154/1154), done.
    remote: Total 3908 (delta 2624), reused 3908 (delta 2624)
    Receiving objects: 100% (3908/3908), 31.89 MiB | 46 KiB/s, done.
    Resolving deltas: 100% (2624/2624), done.
    Richard-Wests-MacBook-Pro:kemia-testing rwest$ cd kemia/
    Richard-Wests-MacBook-Pro:kemia rwest$ open index.html 
    

    Then in Safari Develop > Show Error Console which reports:

    deps.js Failed to load resource: The requested URL was not found on this server.
    images/button-bg.png Failed to load resource: The requested URL was not found on this server.
    

    I notice that the html just has these scripts in the head:

    <script src="build/kemia-whitespace-only.js"></script>
    <script src="kemia/io/json_test_rxn.js"></script>
    <script src="kemia/io/io_test_molsample2.js"></script>
    

    but if I inspect the DOM of the page there's an additional:

    <script type="text/javascript" src="deps.js"></script>
    

    immediately after the line:

    <script src="build/kemia-whitespace-only.js"></script>
    

    Presumably it's coming from this line of kemia-whitespace-only.js:

    if(!goog.global.CLOSURE_NO_DEPS)goog.writeScriptTag_(goog.basePath+"deps.js")}
    

    hence my comment about goog.basePath="" above.

  • paul (at wingu)

    paul (at wingu) August 26th, 2010 @ 03:46 PM

    Thanks for the detailed explanation. I am able to reproduce in Safari and see the error message. You can also see the error when you load http://kemia.github.com/ It shows up in Google Chrome and Firefox too, but it is not so obvious as it is in Safari. This should not have any negative impact on running Kemia, other than the console error message.

    I think the cause is that our build script is trying to include deps.js when it should not. I will take a look at it.

    Thanks!

  • paul (at wingu)

    paul (at wingu) September 9th, 2010 @ 05:06 PM

    • Assigned user set to “paul (at wingu)”
    • Milestone order changed from “4” to “0”
  • paul (at wingu)

    paul (at wingu) September 17th, 2010 @ 12:49 PM

    • Milestone changed from 0.1.0 Clean up user interaction and rendering bugs to 0.9 backlog
    • Milestone order changed from “2” to “0”
  • paul (at wingu)

    paul (at wingu) October 6th, 2010 @ 01:21 PM

    • State changed from “new” to “resolved”

    fixed on reaction editor demo page

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Javascript Reaction Editor and Cheminformatics Library

Shared Ticket Bins

People watching this ticket

Referenced by

Pages