Fixed template generating incorrect charm url

Fixed template generating incorrect charm url

There was also an issue with the view being rendered
before it's container was in the DOM causing a console
error - a check was also put in to abvoid the error

https://code.launchpad.net/~hatch/juju-gui/1157199-charmlink/+merge/154216

(do not edit description out of merge proposal)

[jeff.pihach, 2013-03-19 22:01:08]

Please take a look.

[bac, 2013-03-19 22:11:52]

LGTM with small changes.  I guess that should be LGTMWSC.

https://codereview.appspot.com/7546053/diff/1/app/views/charm.js
File app/views/charm.js (right):

https://codereview.appspot.com/7546053/diff/1/app/views/charm.js#newcode44
app/views/charm.js:44: // If the container node isn't in the dom yet then just
return
Punctuation please.
s/dom/DOM/

https://codereview.appspot.com/7546053/diff/1/app/views/charm.js#newcode45
app/views/charm.js:45: // This avoids a console error caused by out of order
rendering
.

[matthew.scott, 2013-03-19 22:29:33]

LGTM - thanks for the fix.

[jeff.pihach, 2013-03-19 22:33:58]

*** Submitted:

Fixed template generating incorrect charm url

There was also an issue with the view being rendered
before it's container was in the DOM causing a console
error - a check was also put in to abvoid the error

R=bac, matthew.scott
CC=
https://codereview.appspot.com/7546053