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

isVisibleInEditor bug #717

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions _test/types/DecimalTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,8 @@ public function validateSuccessProvider()
['-5.5', '', '4.5'],
['-5,5', '', '4.5'],
['5.5645000', '', ''],
['00055', '', ''],
['-00055', '', ''],
// boundaries
['0', '0', ''],
['0', '', '0'],
Expand Down
4 changes: 2 additions & 2 deletions action/edit.php
Original file line number Diff line number Diff line change
Expand Up @@ -173,8 +173,8 @@ public function makeField(Value $field, $name)
$hide = '';
}

$html = '<div class="field">';
$html .= "<label $hide data-column=\"$colname\" for=\"$id\">";
$html = "<div class='field' $hide>";
$html .= "<label data-column=\"$colname\" for=\"$id\">";
$html .= "<span class=\"label $class\" title=\"$hint\">$trans</span>";
$html .= '</label>';
$html .= "<span class=\"input\">$input</span>";
Expand Down
2 changes: 1 addition & 1 deletion lang/fr/clear_intro.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
**AVERTISSEMENT :** Ceci effacera **toutes** les données qui ont jamais été enregistrées pour ce schéma. Cette opération est irréversible !
**AVERTISSEMENT :** Ceci effacera **toutes** les données enregistrées pour ce schéma. Cette opération est irréversible !
2 changes: 1 addition & 1 deletion lang/fr/delete_intro.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
**AVERTISSEMENT !** Supprimer un schéma supprime certes ce schéma mais aussi **toutes** les données qui lui étaient associées. Cette opération est irréversible !
**AVERTISSEMENT !** Supprimer un schéma supprime le schéma et **toutes** les données qui lui sont associées. Cette opération est irréversible !
2 changes: 1 addition & 1 deletion lang/fr/editor_edit.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

C'est ici que vous ajoutez ou modifiez des champs dans votre schéma.

Chaque champ doit avoir un nom unique. Il est recommandé d'utiliser des noms courts, peu sujets au changement car vous devrez les écrire dans les pages wiki pour afficher vos données. Donnez-leur un nom plus complet (destiné à être vu par les visiteurs) dans la case ''label'' de la configuration du champ.
Chaque champ doit avoir un nom unique. Il est recommandé d'utiliser des noms courts, peu sujets au changement car vous devrez les écrire dans les pages wiki pour afficher vos données. Donnez-leur un nom plus complet (destiné à être vu par les visiteurs) dans la case «label» de la configuration du champ.

Le type de champ détermine quelles données peuvent y être stockées (texte, image, URL...) et influence également la façon dont elles seront affichées plus tard. Chaque type de champ a des options de configuration spécifiques.

Expand Down
2 changes: 1 addition & 1 deletion lang/fr/editor_intro.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
====== Struct - Éditeur de schémas ======

Les schémas sont la base du greffon Struct. Un schéma définit un ensemble de valeurs structurées que l'on peut ensuite affecter à des pages.
Les schémas sont la base du plugin Struct. Un schéma définit un ensemble de valeurs structurées que l'on peut ensuite affecter à des pages.

Choisissez un schéma existant depuis la table des matières ou créez-en un nouveau.
12 changes: 6 additions & 6 deletions lang/fr/lang.php
Original file line number Diff line number Diff line change
Expand Up @@ -38,15 +38,15 @@
$lang['btn_import'] = 'Importer';
$lang['import_warning'] = 'Attention : tous les champs précédemment remplis seront remplacés !';
$lang['del_confirm'] = 'Entrez le nom du schéma pour confirmer sa suppression';
$lang['del_fail'] = 'Les noms de schémas sont différents. Le schéma n\'est pas supprimé.';
$lang['del_fail'] = 'Les noms de schémas sont différents. Le schéma n\'a pas été supprimé.';
$lang['del_ok'] = 'Le schéma a été supprimé.';
$lang['btn_delete'] = 'Supprimer';
$lang['js']['confirmAssignmentsDelete'] = 'Voulez-vous réellement supprimer l\'affectation du schéma "{0}" à la page/catégorie de page "{1}"?';
$lang['js']['actions'] = 'Actions';
$lang['js']['lookup_delete'] = 'Supprimer l\'entrée';
$lang['clear_confirm'] = 'Entrez le nom du schéma pour confirmer l\'effacement de toutes les données.';
$lang['clear_fail'] = 'Les noms de schéma ne correspondent pas. Données non détruites.';
$lang['clear_ok'] = 'Les données du schéma ont été détruites.';
$lang['clear_confirm'] = 'Entrez le nom du schéma pour confirmer la suppression de toutes les données.';
$lang['clear_fail'] = 'Les noms des schémas ne correspondent pas. Aucune donnée n\'a été supprimée.';
$lang['clear_ok'] = 'Les données du schéma ont été supprimées.';
$lang['btn_clear'] = 'effacer';
$lang['tab_edit'] = 'Editer le Schéma';
$lang['tab_export'] = 'Import/Export';
Expand All @@ -65,7 +65,7 @@
$lang['multi'] = 'Vous pouvez entrer plusieurs valeurs séparées par des virgules.';
$lang['multidropdown'] = 'Maintenez les touches CTRL ou ⌘ pour sélectionner plusieurs valeurs.';
$lang['duplicate_label'] = 'Le label <code>%s</code> existe déjà dans ce schéma, la deuxième occurrence a été renommée en <code>%s</code>.';
$lang['emptypage'] = 'Les données du greffon Struct ne sont pas sauvegardées si la page est vide.';
$lang['emptypage'] = 'Les données du plugin Struct ne sont pas sauvegardées si la page est vide.';
$lang['validation_prefix'] = 'Champ [%s] : ';
$lang['Validation Exception Decimal needed'] = 'seuls les décimaux sont autorisés';
$lang['Validation Exception Decimal min'] = 'doit être supérieur ou égal à %d';
Expand All @@ -84,7 +84,7 @@
$lang['Exception nocolname'] = 'Aucun nom de colonne spécifié';
$lang['Exception nolookupmix'] = 'Vous ne pouvez pas agréger plus d\'un schéma de consultation ou le mélanger avec des données de page.';
$lang['Exception No data saved'] = 'Aucune donnée sauvegardée';
$lang['Exception no sqlite'] = 'Le greffon struct requiert le greffon sqlite. Merci de l\'installer et l\'activer.';
$lang['Exception no sqlite'] = 'Le plugin struct requiert le plugin sqlite. Merci de l\'installer et l\'activer.';
$lang['Exception column not in table'] = 'Il n\'y a pas de colonne %s dans le schéma %s.';
$lang['Warning: no filters for cloud'] = 'Les filtres ne sont pas utilisables pour les nuages struct.';
$lang['sort'] = 'Trier selon cette colonne';
Expand Down
8 changes: 4 additions & 4 deletions types/Decimal.php
Original file line number Diff line number Diff line change
Expand Up @@ -99,17 +99,17 @@ public function renderValue($value, \Doku_Renderer $R, $mode)
public function validate($rawvalue)
{
$rawvalue = parent::validate($rawvalue);
$rawvalue = str_replace(',', '.', $rawvalue); // we accept both
$rawvalue = (float) str_replace(',', '.', $rawvalue); // we accept both

if ((string)$rawvalue != (string)(float) $rawvalue) {
if ((string)$rawvalue != (string) $rawvalue) {
throw new ValidationException('Decimal needed');
}

if ($this->config['min'] !== '' && (float) $rawvalue < (float) $this->config['min']) {
if ($this->config['min'] !== '' && $rawvalue < (float) $this->config['min']) {
throw new ValidationException('Decimal min', (float) $this->config['min']);
}

if ($this->config['max'] !== '' && (float) $rawvalue > (float) $this->config['max']) {
if ($this->config['max'] !== '' && $rawvalue > (float) $this->config['max']) {
throw new ValidationException('Decimal max', (float) $this->config['max']);
}

Expand Down