Opened 16 years ago
Closed 16 years ago
#3126 closed Task (fixed)
Rename documentFragment.js to lower case
Reported by: | Frederico Caldeira Knabben | Owned by: | Frederico Caldeira Knabben |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | General | Version: | |
Keywords: | Confirmed Review- | Cc: |
Description
The documentFragment file has been mistakenly named with camel case, just like the class name. It must instead be renamed to full lowercase.
The reference in loader.js is also to be renamed, as well as the test file for it.
Attachments (2)
Change History (6)
Changed 16 years ago by
Attachment: | 3126_part1.patch added |
---|
Changed 16 years ago by
Attachment: | 3126_part2.patch added |
---|
comment:1 Changed 16 years ago by
Keywords: | Review? added |
---|---|
Owner: | set to Artur Formella |
Status: | new → assigned |
comment:2 Changed 16 years ago by
comment:3 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|---|
Owner: | changed from Artur Formella to Frederico Caldeira Knabben |
Status: | assigned → new |
Doing that in this way makes we loose the file log. This must be done in another way, and patches can't be used for that.
I'll do that here directly, as there is no way to get a proper review of it. It's such a small issue that it will should not bring negative consequences.
comment:4 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Both patches are needed to rename those files