Closed
Bug 820770
Opened 12 years ago
Closed 10 years ago
Old CZ plugin script versions are cached across app restarts
Categories
(Other Applications :: ChatZilla, defect)
Other Applications
ChatZilla
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cuteangel24, Assigned: Gijs)
References
Details
(Whiteboard: [cz-0.9.92])
Attachments
(1 file)
1.05 KB,
patch
|
bugzilla-mozilla-20000923
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20100101 Firefox/17.0
Build ID: 20121128204232
Steps to reproduce:
I made a change to one of my CZ scripts in <profile>/chatzilla/scripts/foo/init.js.
Then I restarted chatzilla.
Then I restarted firefox.
Actual results:
The change was ignored. Chatzilla kept using the old script version that didn't even exist on disk anymore.
Expected results:
Chatzilla should have loaded the updated script, especially after I restarted firefox.
Comment 1•11 years ago
|
||
As I understand it, this is a Toolkit but, not one in Firefox or cZ. I remember being subscribed to the issue there, but I can't find the correct bug. If I had, I would have marked this as a duplicate.
Assignee | ||
Comment 2•11 years ago
|
||
Yes, this is a toolkit issue, but I made an opt-in "ignore the cache" thing, and we should change CZ to use it.
Depends on: 821726
Assignee | ||
Updated•11 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: old script versions are cached across firefox restarts → Old CZ plugin script versions are cached across app restarts
Comment 3•11 years ago
|
||
Is that why I'm having difficulties developing my plugin.
Will wait for a fix. Thanks for all the hard work.
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(gijskruitbosch+bugs)
Assignee | ||
Comment 4•10 years ago
|
||
This should be enough, but I've not tested this.
Attachment #8549009 -
Flags: review?(bugzilla-mozilla-20000923)
Assignee | ||
Updated•10 years ago
|
Assignee: rginda → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•10 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #4)
> Created attachment 8549009 [details] [diff] [review]
> should ignore startup cache for /load,
>
> This should be enough, but I've not tested this.
Someone confirmed on IRC that it works for them. :-)
Flags: needinfo?(gijskruitbosch+bugs)
Comment 6•10 years ago
|
||
Comment on attachment 8549009 [details] [diff] [review]
should ignore startup cache for /load,
Review of attachment 8549009 [details] [diff] [review]:
-----------------------------------------------------------------
r=silver, sorry for the delays.
Attachment #8549009 -
Flags: review?(bugzilla-mozilla-20000923) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Whiteboard: [cz-0.9.92]
You need to log in
before you can comment on or make changes to this bug.
Description
•