Opened 9 years ago
Closed 9 years ago
#13782 closed Bug (fixed)
CKEDITOR.warn/error logs are not clear
| Reported by: | Piotrek Koszuliński | Owned by: | Szymon Kupś |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.5.5 |
| Component: | General | Version: | 4.5.4 |
| Keywords: | Cc: |
Description
This is what gets logged:.
[CKEDITOR] range-endcontainer, [browsable object] [CKEDITOR] For more information go to http://docs.ckeditor.com/#!/guide/dev_errors-section-range-endcontainer
The first line is not clear. I think we should print "[CKEDITOR] error code: range-endcontainer, ..."
Or something else that would clarify what the first line means.
Change History (7)
comment:1 Changed 9 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 9 years ago by
| Owner: | set to Szymon Kupś |
|---|---|
| Status: | confirmed → assigned |
comment:3 Changed 9 years ago by
| Status: | assigned → review |
|---|
comment:4 Changed 9 years ago by
| Status: | review → review_failed |
|---|
What I think @Reinmar wanted to emphasize is that it should clearly state that it's a code and I'll second to that. Please use "Error code: " as a prefix (uppercased just like you did, so it will be in line with the next message).
comment:5 Changed 9 years ago by
| Status: | review_failed → review |
|---|
Changed printed label to "Error code:" and updated tests in branch:t/13782.
comment:7 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review → closed |
Note: See
TracTickets for help on using
tickets.
Changed output of CKEDTIOR.warn/error functions to look like this:
Pushed branch:t/13782