Opened 12 years ago
Last modified 12 years ago
#8832 review Bug
Stylesheet parser error if file not on cache
Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Alfonso Martínez de Lizarrondo |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 3.6 |
Keywords: | HasPatch | Cc: | joel.peltonen@… |
Description
#7784 didn't fix the problem correctly
Reproduction steps:
- Clear the cache
- Open the console
- Load http://nightly.ckeditor.com/latest/ckeditor/_samples/stylesheetparser.html
- See the error as reported originally.
Proposed solution:
Revert [6867] and use http://dev.ckeditor.com/attachment/ticket/7784/7784_2.patch as it wasn't rejected because it failed but instead based on coding style. Or rewrite it anyway that you like and test that now it's correctly fixed.
Attachments (1)
Change History (7)
comment:2 Changed 12 years ago by
So anyone know to resolve it ? i can put settimeout if it can help but where ?
comment:5 Changed 12 years ago by
Owner: | set to Alfonso Martínez de Lizarrondo |
---|---|
Status: | confirmed → review |
The patch fixes this problem as well as #7810
comment:6 Changed 12 years ago by
Cc: | joel.peltonen@… added |
---|
in my source of stylesheetparser/plugin.js i dont see code like 7784_2.patch i have this code http://pastebin.com/ycemwBqU , please paste all fixed code with patch 7784_2.patch not only change.
example in patch in code we see "editor.getStylesSet( function( stylesDefinitions ) " in current source code i see obj.getStylesSet = CKEDITOR.tools.override( obj.getStylesSet, function( org )
here http://dev.ckeditor.com/browser/CKEditor/branches/versions/3.6.x/_source/plugins/stylesheetparser/plugin.js?rev=6867 i see version witch i have so how aply patch to this version - patch is for other code ?
so i dont know to witch version is patch like i write please paste to some place working version of this plugin , thanks