Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1501 closed defect (fixed)

The new FireBug.js does not set dojo.hostenv.println

Reported by: attila.lendvai@… Owned by: Dustin Machi
Priority: high Milestone:
Component: General Version: 0.3
Keywords: debug firebug Cc: attila.lendvai@…
Blocked By: Blocking:

Description

the mentioned assignment was probably accidentally removed. due to that the dojo.log.debug stuff is not logging to the console anymore.

dojo.hostenv.println = console.log;

Change History (6)

comment:1 Changed 13 years ago by dylan

Milestone: 0.4
Owner: changed from anonymous to Dustin Machi

comment:2 Changed 13 years ago by attila.lendvai@…

ok, i can see that it's going to be part of a bigger picture. i'd probably introduce a dojo.hostenv.console (.println, .info, .warning, etc)...

but until then, adding back the missing line to console.js would be preferrable to keep the logging stuff go to the console. but if the new stuff is in the works i can of course wait.

thanks!

comment:3 Changed 13 years ago by dylan

Milestone: 0.40.4.1

comment:4 Changed 13 years ago by Dustin Machi

Resolution: fixed
Status: newclosed

Jesse fixed the dojo.log stuff to work with the ff console, though I don't think it was done via println.

comment:5 Changed 13 years ago by attila.lendvai@…

yep, seems like head works again. now my only complaint is that log.debug("foo ", table, " bar") displays only "foo" as text and the rest as an array object. somewhere probably an apply should be used to send all args of the log.debug call to the appropiate console function.

sorry for not sending a patch, laaack of time... :(

comment:6 Changed 12 years ago by (none)

Milestone: 0.4.1

Milestone 0.4.1 deleted

Note: See TracTickets for help on using tickets.