Closed Bug 820770 Opened 7 years ago Closed 5 years ago

Old CZ plugin script versions are cached across app restarts

Categories

(Other Applications :: ChatZilla, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: cuteangel24, Assigned: Gijs)

References

Details

(Whiteboard: [cz-0.9.92])

Attachments

(1 file)

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.
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.
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
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
Is that why I'm having difficulties developing my plugin.
Will wait for a fix. Thanks for all the hard work.
Flags: needinfo?(gijskruitbosch+bugs)
This should be enough, but I've not tested this.
Attachment #8549009 - Flags: review?(bugzilla-mozilla-20000923)
Assignee: rginda → gijskruitbosch+bugs
Status: NEW → ASSIGNED
(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 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+
http://hg.mozilla.org/chatzilla/rev/bca0606b8162
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Whiteboard: [cz-0.9.92]
You need to log in before you can comment on or make changes to this bug.