Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#9428 closed Bug (fixed)

Kurdish: releaser destroys language entries

Reported by: Frederico Caldeira Knabben Owned by: Wiktor Walc
Priority: Normal Milestone: CKEditor 3.6.5
Component: Project : CKReleaser Version:
Keywords: Cc:

Description

The following is a comparison for the "source" language entry in the Kurdish language file (ku.js):

source: 'سه‌رچاوه',    // Original (Good)
source: 'سهرچاوه',    // Processed (Bad)

As we can see, a change happens to the entry after processing. This happens to many other entries as well.

Attachments (2)

for_use_transifex_core_ku_IQ.po (79.8 KB) - added by hozha 7 years ago.
File Core
ku.js (34.3 KB) - added by hozha 7 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by Wiktor Walc

Owner: set to Wiktor Walc
Status: newassigned

Changed 7 years ago by hozha

File Core

comment:2 in reply to:  description Changed 7 years ago by hozha

Hi fredck,

I know what was the problem, the solution just replace ku.jc file that i attached here.

I changed spelling 2 or 3 source, for testing.

Replying to fredck:

The following is a comparison for the "source" language entry in the Kurdish language file (ku.js):

source: 'سه‌رچاوه',    // Original (Good)
source: 'سهرچاوه',    // Processed (Bad)

As we can see, a change happens to the entry after processing. This happens to many other entries as well.

Last edited 7 years ago by hozha (previous) (diff)

Changed 7 years ago by hozha

Attachment: ku.js added

comment:3 Changed 7 years ago by Wiktor Walc

CKLangTool fixed with [7637], CKReleaser with [7642].

@hozha - could you confirm that the nightly build is fine now?

comment:4 in reply to:  3 Changed 7 years ago by hozha

Hello again,

Yes now working perfect.

i just changed a few spelling with the new Kurdish writer, and now looking perfect. the problem was, i used a old Kurdish writer, after i updated the Kurdish writer and changed a few spelling source, now looking perfect for the source that i changed it today.

I will fix all source that content (هـ).

best regards, Hozha.

Replying to wwalc:

CKLangTool fixed with [7637], CKReleaser with [7642].

@hozha - could you confirm that the nightly build is fine now?

comment:5 Changed 7 years ago by Wiktor Walc

What I did in fact was leaving Persian and Kurdish language files unminified in release version. The files are a big larger, but still with an acceptable size. It means that nothing should be destroyed now.

Just to make sure - the whole Kurdish interface for CKEditor seems to be fine now in the nightly build, is that correct?


When I was trying to deal with a minified version of language file, I did a quick check earlier with the language file provided here (and the one downloaded from Transifex), and for example link.charset was still being destroyed in a compiled version of language file.

Is that particular entry (link.charset) supposed to be fixed already in ku.js attached here?

comment:6 Changed 7 years ago by Wiktor Walc

Resolution: fixed
Status: assignedclosed

Marking as fixed, files are no longer minified so they will definitely not be destroyed.

comment:7 in reply to:  5 Changed 7 years ago by hozha

for me Kurdish interface for CKEditor looks terrific now in the nightly build, Perfect.

So, that means I do not need to change the source that content (ه), becuase now everything looks fine.

I test now, it seems fine.

Replying to wwalc:

What I did in fact was leaving Persian and Kurdish language files unminified in release version. The files are a big larger, but still with an acceptable size. It means that nothing should be destroyed now.

Just to make sure - the whole Kurdish interface for CKEditor seems to be fine now in the nightly build, is that correct?


When I was trying to deal with a minified version of language file, I did a quick check earlier with the language file provided here (and the one downloaded from Transifex), and for example link.charset was still being destroyed in a compiled version of language file.

Is that particular entry (link.charset) supposed to be fixed already in ku.js attached here?

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy