SortAscending, SortDescending での改行の扱いを修正

• Informador: (Anónimo)
• Propietario: (Ninguno)
• Tipo: Bugs
• Estado: Cerrado
• Componente: (Ninguno)
• Hito: (Ninguno)
• Prioridad: 4
• Gravedad: 5 - Medium
• Resolución: Fixed
• Fichero: Ninguno

Details Responder

editor addin.sct で、昇順ソート・降順ソートの実行時、CRLFの途中で分割されていたのを修正してみました。

• 文字列が「\n」で分割されていたのを、「\r\n」で分割するように修正
  
• 選択範囲の末尾が改行まで含んでいるか、改行の手前までを選択しているかで挙動が異なっていたのを修正
  

 function SortAscending(Text) {
         var lines = Text.split("\r\n");
 
         if (lines.length == 1) {
                 return Text;
         } else if (lines[lines.length - 1] == "") {
                 lines.pop();
                 return lines.sort().join("\r\n") + "\r\n";
         } else {
                 return lines.sort().join("\r\n");
         }
 }
 
 function SortDescending(Text) {
         var lines = Text.split("\r\n");
 
         if (lines.length == 1) {
                 return Text;
         } else if (lines[lines.length - 1] == "") {
                 lines.pop();
                 lines.sort(function(a, b) { return a < b ? 1 : -1; });
                 return lines.join("\r\n") + "\r\n";
         } else {
                 return lines.sort(function(a, b) { return a < b ? 1 : -1; }).join("\r\n");
         }
 }

Ticket History (3/5 Histories)

2020-03-19 16:13 Updated by: None

• New Ticket "SortAscending, SortDescending での改行の扱いを修正" created

2020-03-20 00:22 Updated by: None

Comentario Responder

修正ありがとうございます。

var lines = Text.split("\r\n"); だと 改行文字がUnixタイプのファイル("\n"のみ)の場合にうまくいかないように思いますので、若干変更して取り込ませていただきます。

2020-03-23 11:34 Updated by: None

Comentario Responder

Reply To (Anonymous)

修正ありがとうございます。 var lines = Text.split("\r\n"); だと 改行文字がUnixタイプのファイル("\n"のみ)の場合にうまくいかないように思いますので、若干変更して取り込ませていただきます。

お手数おかけします。
SortDescending で区切りが "\r\n" のままになっているようです。

 function SortDescending(Text) { 
         var eol = Text.match(/\r\n|\n|\r/);
         var lines = Text.split("\r\n");   //←←←
 
         //………
 }

2020-03-24 07:17 Updated by: None

Comentario Responder

ありがとうございます。修正します。

2020-04-29 10:02 Updated by: sdottaka

• Resolución Update from Ninguno to Fixed
• Estado Update from Open to Cerrado