[PR #8713] [MERGED] Исправлена сортировка названий файлов стратегий по номерам #223

Closed
opened 2026-07-19 00:42:27 +03:00 by mirror-bot · 0 comments

📋 Pull Request Information

Original PR: https://github.com/Flowseal/zapret-discord-youtube/pull/8713
Author: @JustChickNugget
Created: 1/5/2026
Status: ✅ Merged
Merged: 1/7/2026
Merged by: @Flowseal

Base: main ← Head: ux/fix-sorting


📝 Commits (1)

  • ece85ec Fixed strategy files sorting by numbers

📊 Changes

2 files changed (+5 additions, -8 deletions)

View changed files

📝 service.bat (+4 -7)
📝 utils/test zapret.ps1 (+1 -1)

📄 Description

Так как Windows по умолчанию использует сортировку в формате ASCIIbetical, то пользователи могли заметить то, что "ALT2" идёт после "ALT11", а не после "ALT1". Поэтому я решил попробовать исправить это, но немножко "подвинуть" такую сортировку в правильном направлении, используя Regex. То есть, это до сих пор не является сортировкой в алфавитном порядке.

Был вариант с переименованием самих файлов стратегий, но я подумал, что это может быть не самой лучшей идеей в плане дальнейшей поддержки стратегий (но я не сильно в это углублялся, поэтому точно не скажу). Была идея просто добавить нолик слева от номера стратегии, если этот самый номер был по значению меньше 10, то есть: "ALT01", "ALT02", "ALT10", "ALT11". Этот вариант бы сработал, так как в ASCII 0 < 1. Его я примерно и реализовал в скриптах.

Единственная проблема текущей реализации - из-за вызова PowerShell будет заметно, что названия выводятся дольше.

Я бы хотел узнать у @Flowseal, если переименовывать файлы можно. А пока я остановился на варианте с изменением логики сортировки в скриптах service.bat и utils/test zapret.ps1.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Flowseal/zapret-discord-youtube/pull/8713 **Author:** [@JustChickNugget](https://github.com/JustChickNugget) **Created:** 1/5/2026 **Status:** ✅ Merged **Merged:** 1/7/2026 **Merged by:** [@Flowseal](https://github.com/Flowseal) **Base:** `main` ← **Head:** `ux/fix-sorting` --- ### 📝 Commits (1) - [`ece85ec`](https://github.com/Flowseal/zapret-discord-youtube/commit/ece85ec82175f24d427d872fd2d8ddf601f3f1fd) Fixed strategy files sorting by numbers ### 📊 Changes **2 files changed** (+5 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `service.bat` (+4 -7) 📝 `utils/test zapret.ps1` (+1 -1) </details> ### 📄 Description Так как Windows по умолчанию использует сортировку в формате ASCIIbetical, то пользователи могли заметить то, что "ALT2" идёт после "ALT11", а не после "ALT1". Поэтому я решил попробовать исправить это, но немножко "подвинуть" такую сортировку в правильном направлении, используя Regex. То есть, это до сих пор не является сортировкой в алфавитном порядке. Был вариант с переименованием самих файлов стратегий, но я подумал, что это может быть не самой лучшей идеей в плане дальнейшей поддержки стратегий (но я не сильно в это углублялся, поэтому точно не скажу). Была идея просто добавить нолик слева от номера стратегии, если этот самый номер был по значению меньше 10, то есть: "ALT01", "ALT02", "ALT10", "ALT11". Этот вариант бы сработал, так как в ASCII `0` < `1`. Его я примерно и реализовал в скриптах. Единственная проблема текущей реализации - из-за вызова PowerShell будет заметно, что названия выводятся дольше. Я бы хотел узнать у @Flowseal, если переименовывать файлы можно. А пока я остановился на варианте с изменением логики сортировки в скриптах `service.bat` и `utils/test zapret.ps1`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror-bot added the pull-request label 2026-07-19 00:42:27 +03:00
mirror-bot changed title from [PR #8713] [MERGED] Исправлена сортировка названий файлов стратегий по номерам to [PR #8713] Исправлена сортировка названий файлов стратегий по номерам 2026-07-20 04:21:52 +03:00
mirror-bot changed title from [PR #8713] Исправлена сортировка названий файлов стратегий по номерам to [PR #8713] [MERGED] Исправлена сортировка названий файлов стратегий по номерам 2026-07-20 23:19:04 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Utilities/zapret-discord-youtube#223