Redmine

Be careful if later you want to use other javascript lib than the ones packaged with rails. I had surprises where some libs are requesting prototype, and they were not able to detect it any more with the cache mode.
To explain the cache mode in one line, all js files are concatened in one single file, easily compressable and cacheable.

Thanks

For me this Patch wont work.
If I've applied it there are no actions on the comment buttons of the issues, also the context menu at the issue list will not be available.
If I've removed it all the things works fine.

Philippe Lafoucrière wrote:

Be careful if later you want to use other javascript lib than the ones packaged with rails. I had surprises where some libs are requesting prototype, and they were not able to detect it any more with the cache mode.

Can you provide an example? I still see the Prototype object:

>>> Prototype.Version
"1.6.0.1" 

Jan Losinski wrote:

For me this Patch wont work.
If I've applied it there are no actions on the comment buttons of the issues, also the context menu at the issue list will not be available.
If I've removed it all the things works fine.

Can you provide me your browser version and Operating system version? The context menu and issue links are working fine for me in Firefox. Also make sure you've cleared the browser cache.

I know starbox (http://www.nickstakenburg.com/projects/starbox/) at least won't work in this case (all js compacted in one all.js file).

Thx
Philippe

Eric Davis wrote:

Can you provide me your browser version and Operating system version? The context menu and issue links are working fine for me in Firefox. Also make sure you've cleared the browser cache.

The cache was cleared, the Problem exist until I remove :cache => true
Browser: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1) Gecko/2008071618 Iceweasel/3.0.1 (Debian-3.0.1-1)
Rails: 2.1.0
Ruby: 1.8.2
libjs-prototype: 1.6.0.2

Rails runs in a apache-fcgid environment.

Jan Losinski wrote:

The cache was cleared, the Problem exist until I remove :cache => true
Browser: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1) Gecko/2008071618 Iceweasel/3.0.1 (Debian-3.0.1-1)
Rails: 2.1.0
Ruby: 1.8.2
libjs-prototype: 1.6.0.2

Rails runs in a apache-fcgid environment.

I tested with the exact same browser and the context menu is working for me. In fact the context menu isn't included in the cached JavaScript so it shouldn't be affected at all.

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1) Gecko/2008071618 Iceweasel/3.0.1 (Debian-3.0.1-1)

• Wiki formatting buttons disappear (for project descriptions, issues, comments...)
• The preview link does not work (when creating or commenting an issue)
• Issue filtering does no work: parts of filter expressions are missing, adding filters does not work, listing all issues (including closed) does not work, ...

I tried migrating the db, clearing the redmine cache and clearing the browser cache with no result.

Removing ":cache => true" puts everything back to normal.

Rails 2.1.0, ruby 1.8.5
Browser: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1) Gecko/2008072820 Firefox/3.0.1

I think the problem is due to the fact that your application can't write all.js in /public/javascripts folder because the app doesn't have write access on this repository. And thus, all javascripts are unavailable.

I personnally use this cache technique for a while here on redmine.org but everyone may not want to give write access to the app on the /public folder. So IMHO, this patch should be reverted.

Jean-Philippe Lang wrote:

I think the problem is due to the fact that your application can't write all.js in /public folder because the app doesn't have write access on this repository.

On my Box, all.js exist in /public/javascripts and RoR can write to it, but it's an empty file.

Ok, seeing all the problems this patch is raising, and the very little improvement, we can clearly reject the patch. Sorry for that guys.

Eric Davis wrote:

I've reverted the commit. Can everyone who had an issue please update to r1771 and see if the problem is reverted. Sorry for the hassle.

If the problem may occur on some installations, and is server side configuration issue, it does not make the feature proposed unusable for everybody. Can't it just be let as an option for site administrator ?

It looks like r1786 turned this back on somehow and my installation is broken. Can we get this turned back off please or is there some other fix in place that allows it to be toggled on/off?

Has this been turned on again in rev1794? I was just wondering what those bugs were, which occured every now and then, and disappeared after reloading. I had to change the folder permissions for "javascripts" and now Redmine is working with JS support again.

Richard Hurt wrote:

It looks like r1786 turned this back on somehow and my installation is broken. Can we get this turned back off please or is there some other fix in place that allows it to be toggled on/off?

The merge from the hooks branch in r1786 pulled in the bug. If you update to r1795, I've backed it out.

Eric Davis wrote:

Richard Hurt wrote:

It looks like r1786 turned this back on somehow and my installation is broken. Can we get this turned back off please or is there some other fix in place that allows it to be toggled on/off?

The merge from the hooks branch in r1786 pulled in the bug. If you update to r1795, I've backed it out.

Thanks, that did the trick.

Later...
Richard