Closed
Bug 80125
Opened 24 years ago
Closed 22 years ago
Logging facility in Chatzilla.
Categories
(Other Applications :: ChatZilla, enhancement, P2)
Other Applications
ChatZilla
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: anthony.vickers, Assigned: rginda)
References
Details
Attachments
(1 file, 3 obsolete files)
(deleted),
patch
|
rginda
:
review+
|
Details | Diff | Splinter Review |
Would be nice to be able to log chats and server info to seperate named and
dated files.
Updated•24 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Comment 1•23 years ago
|
||
Also having the option to save as plain text or html would be nice.
Updated•23 years ago
|
Blocks: patchmaker
Updated•23 years ago
|
No longer blocks: patchmaker
you can copy and paste out of the Chatzilla window (no line breaks or colors tho')
Assignee | ||
Comment 4•23 years ago
|
||
If you paste into composer you'll get the html. You can then link in the CSS of
your choice for colors.
What about an option for mirc-like logs (text file with timestamps) ?
It would be useful for use with stats programs like mircstats...
or if it's stored in some form of xml maybe it would be possible to generate
some kind of stats with mozilla itself and publish it regularly on some http
server? :)
Comment 6•23 years ago
|
||
*** Bug 119174 has been marked as a duplicate of this bug. ***
Comment 7•23 years ago
|
||
Remove myself from QA of 33 open Chatzilla bugs and change to default QA
contact, since I have no way to verify these easily. Still no working Mozilla
on my primary platform and it doesn't look like it will happen anytime soon. :(
QA Contact: mozilla → samuel
Comment 8•22 years ago
|
||
Automatic logging would also help in the event of a mozilla or computer crash.
Comment 9•22 years ago
|
||
I'll try to implement this feature.
(a) Create "Chat" as a subdirectory of the profile directory.
(b) Create a log file when a new view is opened and write every message
to it with timestamp.
(c) Default to off.
Is this OK?
Comment 10•22 years ago
|
||
I copied file-utils.js from Venkman. It's not included in the patch, but I
made a one-line change to it which is at the end of the patch.
Updated•22 years ago
|
Attachment #123816 -
Flags: review?(rginda)
Assignee | ||
Comment 11•22 years ago
|
||
Comment on attachment 123816 [details] [diff] [review]
first draft
+ switch (view.TYPE)
+ {
+ case "IRCNetwork":
+ view.logFile = view.name.replace(/:.*/, "") + ".log";
+ break;
when do we have a : character in a network name?
+ }
+ view.logFile = view.logFile.replace(/\//g, "%2f").replace(/:/g,
"%3a");
+ view.logFile = dir + "/" + view.logFile;
that's not cross platform, you need to call .append(view.logFile) on the
original object returned from getSpecialDirectory().
+ try
+ {
+ view.logFile.close();
+ }
+ catch (ex)
+ {
+ }
why the empty catch clause? what would view.logFile.close() throw?
+ try
+ {
+ if (this.prefBranch.getBoolPref("logging"))
+ client.onInputLog({inputData: "on"});
+ }
+ catch (ex)
+ {
+ }
This pattern appears in a few places, how about adding some utility functions
in readprefs.js to take care of safely reading a pref?
@@ -2364,6 +2373,7 @@
{
var canMergeData = false;
var canCollapseRow = false;
+ var logText = "";
I'd guess most long-term log users will want each message to start with a
date/time stamp. If you add that date/time stamp as an attribute of the
msg-user TD, users will finally be able to write CSS with a timestamp on every
line.
Attachment #123816 -
Flags: review?(rginda) → review-
Comment 12•22 years ago
|
||
Attachment #123816 -
Attachment is obsolete: true
Updated•22 years ago
|
Attachment #125273 -
Flags: review?(rginda)
Updated•22 years ago
|
Attachment #125273 -
Attachment is obsolete: true
Attachment #125273 -
Flags: review?(rginda)
Comment 13•22 years ago
|
||
Updated•22 years ago
|
Attachment #125375 -
Flags: review?(rginda)
Comment 14•22 years ago
|
||
Attachment #125375 -
Attachment is obsolete: true
Updated•22 years ago
|
Attachment #125375 -
Flags: review?(rginda)
Updated•22 years ago
|
Attachment #125559 -
Flags: review?(rginda)
Assignee | ||
Comment 15•22 years ago
|
||
I just posted the xpi for the latest patch to
http://www.hacksrus.com/~ginda/chatzilla/
Assignee | ||
Updated•22 years ago
|
Attachment #125559 -
Flags: review?(rginda) → review+
Assignee | ||
Comment 16•22 years ago
|
||
marking fixed, 0.8.34 checked in.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Core → Other Applications
You need to log in
before you can comment on or make changes to this bug.
Description
•