Closed
Bug 110450
Opened 23 years ago
Closed 23 years ago
javascript strict warnings in EditorPublish.js
Categories
(SeaMonkey :: Composer, defect)
SeaMonkey
Composer
Tracking
(Not tracked)
People
(Reporter: bugzilla, Assigned: cmanske)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Brade
:
review+
|
Details | Diff | Splinter Review |
Warning: reference to undefined property gDialog.SiteList
Source File: chrome://editor/content/EditorPublish.js
Line: 295
Warning: reference to undefined property gDialog.PageTitleInput
Source File: chrome://editor/content/EditorPublish.js
Line: 289
Warning: reference to undefined property gDialog.TabPanels
Source File: chrome://editor/content/EditorPublish.js
Line: 335
build 20011115
This happens in a debug build by doing what?
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•23 years ago
|
||
start Compose
select File -> Publish -> Publish
remember to have js strict warnings set to on via prefs -> debug
Comment 3•23 years ago
|
||
I think the three above strict JS warnings are no longer in effect. Note in
http://lxr.mozilla.org/seamonkey/source/editor/ui/dialogs/content/EditorPublish.js
lines 50, 55, and 57.
In build Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.7+) Gecko/20020105 I get:
Warning: test for equality (==) mistyped as assignment (=)?
Source File: chrome://editor/content/EditorPublish.js
Line: 160, Column: 26
Source Code:
if (len = urlLen)
Patch for this strict warning coming up.
Comment 4•23 years ago
|
||
Comment 6•23 years ago
|
||
brade: can you take a look at Alex's patch?
Comment 7•23 years ago
|
||
Comment on attachment 63704 [details] [diff] [review]
patch for comparison instead of assignment strict warning
r=brade
Attachment #63704 -
Flags: review+
Comment 8•23 years ago
|
||
-->cmanske who has this change in his tree already
Assignee: syd → cmanske
Status: ASSIGNED → NEW
Target Milestone: Future → ---
Assignee | ||
Comment 9•23 years ago
|
||
Yep, I have this in my latest work for bug 88208
*** This bug has been marked as a duplicate of 88208 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Comment 11•23 years ago
|
||
Henrik, can you test this out using 3/8 build...fix for 88208 went
into this build, so you should not see this problem anymore..
Please verify...and REOPEN this bug if you still see the problem..thanks!
Reporter | ||
Comment 12•23 years ago
|
||
v dupe. I cant get any strict warn anymore...
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•