Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Permettre le tri des chansons #36

Closed
paternal opened this issue May 15, 2014 · 3 comments
Closed

Permettre le tri des chansons #36

paternal opened this issue May 15, 2014 · 3 comments
Assignees
Milestone

Comments

@paternal
Copy link
Contributor

Le tri des chansons ne fonctionne plus depuis a715c8f. Avec l'extension de la syntaxe du contenu d'un songbook (#9 et d83be38), se pose la question de la manière de trier. Je vois plusieurs possibilités :

  • on enlève cette fonctionnalité : les chansons sont ajoutées dans l'ordre dans lequel elles sont ajoutées dans le fichier .sb ;
  • on trie toutes les chansons placées entre deux sections. Le problème est que si on étend les types de contenu pouvant être inclus (par exemple des image, comme proposé en exemple en Utilisation d'un moteur de templates #9), il va falloir gérer entre quel type de contenu il faut trier, et selon quelles clefs ;
  • on ne trie qu'au sein des regexp : si un fichier contient ["tagada/*.sg", "tsointsoin/*sg"], les chansons correspendant à la première regexp sont triées d'une part, puis celles de la seconde, mais elles ne sont pas mélangées.

Ma préférence va pour la troisième solution, sachant qu'elle pourra être étendue avec un type de contenu sorted, qui triera l'ensemble des chansons qu'il contient : [["section", "Traditionnel"], ["sorted", "tagada/*.sb", "tsointsoin/*.sb"]].

@paternal paternal added this to the 4.0 milestone May 15, 2014
@paternal paternal self-assigned this May 15, 2014
@Luthaf
Copy link
Contributor

Luthaf commented May 15, 2014

S'il faut vraiment trier, je suis pour la troisième proposition aussi. Mais est-ce vraiment utile ? Dans le pire des cas, les chansons d'une regex seront dans le désordre et voila tout.

@paternal
Copy link
Contributor Author

En fait, en y repensant, je penche plutôt, pour plus de cohérence, que les expressions régulières sont incluses dans l'ordre alphabétique de noms de fichiers, et on ajoutera un type sorted si on veut trier les chansons.

Je marque ça à ma todo liste (mais vu ma disponibilité en ce moment, vous avez le temps de faire des commentaires avant que je m'y mette).

@paternal
Copy link
Contributor Author

Fait : 0a4d763

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants