Index: extensions/event_cats/admin/autolog_entries.inc.php =================================================================== --- extensions/event_cats/admin/autolog_entries.inc.php (revision 4091) +++ extensions/event_cats/admin/autolog_entries.inc.php (revision 4092) @@ -88,8 +88,4 @@ switch ($ec_entry['action']) { - case 'ec_ok_f_pb' : - case 'ec_ok_f_pb_img_pb' : - $page['errors'][$ec_entry['code']] = - $ec_entry['code'].' : '.l10n('ec_forced_pb'); case 'ec_ok' : case 'ec_ok_img_pb' : @@ -103,8 +99,5 @@ case 1: // Category case 3: // Image - $url_end = ( - $ec_entry['forced'] == 'true' and - !is_in($ec_entry['action'], '_f_pb') - ) ? '' : '&cat='.$arg1; + $url_end = ($ec_entry['forced'] == 'true') ? '' : '&cat='.$arg1; $dspl = l10n('ec_dspl_ok_cat_pre'). @@ -148,8 +141,5 @@ break; case 2: // Additional Page - $url_end = ( - $ec_entry['forced'] == 'true' and - !is_in($ec_entry['action'], '_f_pb') - ) ? '' : '&ap='.$arg2; + $url_end = ($ec_entry['forced'] == 'true') ? '' : '&ap='.$arg2; $dspl = ($ec_ap_ok) ? l10n('ec_dspl_ok_ap_pre'). @@ -182,7 +172,4 @@ break; - case 'ec_ok_f_pb_ap_pb' : - $page['errors'][$ec_entry['code']] = - $ec_entry['code'].' : '.l10n('ec_forced_pb'); case 'ec_nok_ap_pb' : // No need to check here if Additional Pages is case 'ec_ok_ap_pb' : // active : we can arrive here only when it is. @@ -190,8 +177,5 @@ $ec_entry['code'].' : '.l10n('ec_add_page_miss'); if (is_in($ec_entry['action'], 'ec_ok')) { - $url_end = ( - $ec_entry['forced'] == 'true' and - !is_in($ec_entry['action'], '_f_pb') - ) ? '' : '&ap='.$arg2; + $url_end = ($ec_entry['forced'] == 'true') ? '' : '&ap='.$arg2; $dspl = l10n('ec_dspl_ok_home'); } @@ -208,12 +192,6 @@ break; - case 'ec_ok_f_pb_cat_pb' : - $page['errors'][$ec_entry['code']] = - $ec_entry['code'].' : '.l10n('ec_forced_pb'); case 'ec_ok_cat_pb' : - $url_end = ( - $ec_entry['forced'] == 'true' and - !is_in($ec_entry['action'], '_f_pb') - ) ? '' : '&cat='.$arg1; + $url_end = ($ec_entry['forced'] == 'true') ? '' : '&cat='.$arg1; $page['errors'][] = $ec_entry['code'].' : '.l10n('ec_category_miss'); @@ -283,5 +261,4 @@ 'code_URL' => $code_URL, 'log_OK' => $log_OK, - 'f_pb' => is_in($ec_entry['action'], '_f_pb'), ); } Index: extensions/event_cats/admin/autolog_new.inc.php =================================================================== --- extensions/event_cats/admin/autolog_new.inc.php (revision 4091) +++ extensions/event_cats/admin/autolog_new.inc.php (revision 4092) @@ -62,11 +62,10 @@ if ( is_in($ec_entry['action'], 'ec_ok') and - !is_in($ec_entry['action'], '_f_pb') and $ec_entry['forced'] == 'false' ) { $t[$ec_entry['id']] = $ec_entry['code']; - $ec_lists['duplicable_codes'][$ec_entry['code']]['id'] = + $ec_lists['duplicable_codes']['codes'][$ec_entry['code']]['id'] = $ec_entry['id']; - $ec_lists['duplicable_codes'][$ec_entry['code']]['user_id'] = + $ec_lists['duplicable_codes']['codes'][$ec_entry['code']]['user_id'] = $ec_entry['user_id']; } @@ -74,10 +73,10 @@ foreach ($t as $ec_id => $ec_code) { $ec_lists['duplicable_codes']['ids'][$ec_id] = - $ec_lists['duplicable_codes'][$ec_code]['id']; + $ec_lists['duplicable_codes']['codes'][$ec_code]['id']; } $ec_def_vals = array(); -$ec_default_action = 'create'; +$ec_default_action = 'create_ask'; $ec_default_entry = '-1'; $ec_def_auto_code_dg_nb = read_ec_conf('auto_code_dg_nb'); @@ -93,6 +92,9 @@ ) { switch ($_POST['ec_act1']) { + case 'duplicate_entry_submit': // submit failed + case 'modify_entry_submit': // submit failed case 'duplicate_entry_ask': case 'modify_entry_ask': + case 'create_ask': case 'disable': case 'create': @@ -118,7 +120,10 @@ (count($ec_lists['user_ids']) == 0) ? 'style = "display:none;"' : '' ); -$template->assign('ec_is_creation', - ($ec_default_action == 'duplicate_entry_ask' or - $ec_default_action == 'create') +$template->assign('ec_is_creation', ( + $ec_default_action == 'duplicate_entry_submit' or + $ec_default_action == 'duplicate_entry_ask' or + $ec_default_action == 'create_ask' or + $ec_default_action == 'create' + ) ); $template->assign('ec_default_action',$ec_default_action); Index: extensions/event_cats/admin/config.inc.php =================================================================== --- extensions/event_cats/admin/config.inc.php (revision 4091) +++ extensions/event_cats/admin/config.inc.php (revision 4092) @@ -0,0 +1,3 @@ + Index: extensions/event_cats/admin/duplication.inc.php =================================================================== --- extensions/event_cats/admin/duplication.inc.php (revision 4091) +++ extensions/event_cats/admin/duplication.inc.php (revision 4092) @@ -0,0 +1,3 @@ + Index: extensions/event_cats/admin/evntcats_admin.php =================================================================== --- extensions/event_cats/admin/evntcats_admin.php (revision 4091) +++ extensions/event_cats/admin/evntcats_admin.php (revision 4092) @@ -73,4 +73,7 @@ ) change_ec_conf('auto_code_dg_nb', $_POST['ec_in_up_auto_code_length']); +$ec_howto = (read_ec_conf('howto') == '0'); +if ($ec_howto) change_ec_conf('howto', '1'); + // +-----------------------------------------------------------------------+ // | Tabsheet @@ -88,7 +91,7 @@ if (isset($_POST['ec_act1'])) { - if ( /* - $_POST['ec_act1'] == 'autolog_new' or // ? */ + if ( $_POST['ec_act1'] == 'duplicate_entry_ask' or + $_POST['ec_act1'] == 'create_ask' or $_POST['ec_act1'] == 'disable' or $_POST['ec_act1'] == 'modify_entry_ask' @@ -146,5 +149,6 @@ */ -$template->assign('ec_howto', (count($page['errors']) == 0 and count($page['infos']) == 0)); +$template->assign('ec_howto', + ($ec_howto and (count($page['errors']) ==0 and count($page['infos']) == 0))); $template->assign('EVNTCATS_PATH',EVNTCATS_PATH); Index: extensions/event_cats/admin/evntcats_theme.css =================================================================== --- extensions/event_cats/admin/evntcats_theme.css (revision 4091) +++ extensions/event_cats/admin/evntcats_theme.css (revision 4092) @@ -68,2 +68,6 @@ color:green; } + +img.button { + vertical-align:middle; +} Index: extensions/event_cats/admin/template/autolog_entries.js =================================================================== --- extensions/event_cats/admin/template/autolog_entries.js (revision 4091) +++ extensions/event_cats/admin/template/autolog_entries.js (revision 4092) @@ -3,32 +3,58 @@ /* déplacer les liens de test et de suppression en-dessous de la table (en-dessous du champ "URL"), éventuellement en les accompagnant des -icônes ; et y rajouter le lien de modification (envoyer $_POST{['ec_act1'] = 'modify_entry_ask' et $_POST{['ec_entry_sel'] = <# entré concernée>) +icônes ; et y rajouter le lien de modification (envoyer $_POST{['ec_act1'] = 'modify_entry_ask' et $_POST{['ec_entry_sel'] = <# entrée concernée>) */ -var ec_checked_line = 0; +var ec_checked_line = -1; var ec_JS_message = new Array(); function ec_mark_line(ec_id) { + var t = ''; ec_checked_line = ec_id; - if (ec_checked_line == 0) { + if (ec_checked_line < 0) { document.getElementById('selection--1').checked = 'checked'; document.getElementById('selection--1').checked = ''; - document.getElementById('ec_show_URL').value = ''; + document.getElementById('ec_show_URL').value = ''; + document.getElementById('ec_table_URL').href = 'javascript:void(0);'; } else { + t = document.getElementById('ec_url_id_' + ec_checked_line).innerHTML; document.getElementById("selection-" + ec_checked_line).checked = 'checked'; - document.getElementById('ec_show_URL').value = - (ec_checked_line < 0) ? '' : - document.getElementById('ec_table_URL_' + ec_checked_line).href; + document.getElementById('ec_show_URL').value = t; + document.getElementById('ec_table_URL').href = t; } } -function ec_check_d(ec_id, ec_act) { +function ec_check_ds(ec_id) { ec_mark_line(ec_id); - if (confirm(ec_JS_message['ec_confirm_' + ec_act])) { - document.getElementById('ec_act1_id').value = ec_act; + document.getElementById('ec_act1_id').value = 'disable'; + document.forms['ec_view_codes'].submit(); +} + +function ec_check_dl() { + if (ec_checked_line >= 0) { + if (confirm(ec_JS_message['ec_confirm_delete'])) { + document.getElementById('ec_act1_id').value = 'delete'; + document.forms['ec_view_codes'].submit(); + } + } + else alert(ec_JS_message['ec_clk_entry_miss']); +} + +function ec_check_m() { + if (ec_checked_line >= 0) { + document.getElementById('ec_act1_id').value = 'modify_entry_ask'; document.forms['ec_view_codes'].submit(); } + else alert(ec_JS_message['ec_clk_entry_miss']); +} + +function ec_check_tst() { + if (ec_checked_line < 0) { + alert(ec_JS_message['ec_clk_entry_miss']); + return false; + } + else return true; } @@ -49,9 +75,4 @@ } -function ec_check_tst(ec_id) { - ec_mark_line(ec_id); - return confirm(ec_JS_message['ec_confirm_test']); -} - function ec_test() { alert('OK.'); @@ -59,6 +80,6 @@ function ec_init() { - ec_checked_line = 0; + ec_checked_line = -1; document.getElementById('ec_act1_id').value = ''; - ec_mark_line(0); + ec_mark_line(-1); } Index: extensions/event_cats/admin/template/autolog_entries.tpl =================================================================== --- extensions/event_cats/admin/template/autolog_entries.tpl (revision 4091) +++ extensions/event_cats/admin/template/autolog_entries.tpl (revision 4092) @@ -24,5 +24,5 @@
+ | + {'ec_lower_tools_desc'|@translate} + | +
+ + + + + {'ec_test_text'|@translate} + + + + + {'ec_delete_text'|@translate} + + + + + {'ec_modify_text'|@translate} + |
{if $ec_is_creation} @@ -62,6 +67,4 @@ |
{if $ec_is_creation}
-
- {* *}
@@ -293,4 +294,5 @@ ec_JS_message['ec_new_pwd_gen'] = "{'ec_new_pwd_gen'|@translate|@escape:javascript}"; ec_JS_message['ec_act_entry_miss'] = "{'ec_act_entry_miss'|@translate|@escape:javascript}"; + ec_JS_message['ec_confirm_del_other'] = "{'ec_confirm_del_other'|@translate|@escape:javascript}"; ec_JS_message['ec_up_code_lbl_home'] = "{'ec_up_code_lbl_home'|@translate}"; ec_JS_message['ec_submit_code_miss'] = "{'ec_submit_code_miss'|@translate}"; Index: extensions/event_cats/admin/template/header.inc.tpl =================================================================== --- extensions/event_cats/admin/template/header.inc.tpl (revision 4091) +++ extensions/event_cats/admin/template/header.inc.tpl (revision 4092) @@ -21,7 +21,8 @@ {'ec_admin_page_howto'|@translate} + + + +{else} + {/if} - - - - Index: extensions/event_cats/include/ec_conf.inc.php =================================================================== --- extensions/event_cats/include/ec_conf.inc.php (revision 4091) +++ extensions/event_cats/include/ec_conf.inc.php (revision 4092) @@ -33,11 +33,13 @@ _ the number of digits when creating a new code automatically ($ec_conf[2] <=> read_ec_conf('auto_code_dg_nb')) ; + _ whether the "how to" banner has already been displayed or not + ($ec_conf[3] <=> read_ec_conf('howto')) ; _ what to do in case an unknown code is used with "autolog" parameter - ($ec_conf[3] <=> read_ec_conf('unknown_code')) : + ($ec_conf[4] <=> read_ec_conf('unknown_code')) : 0 : nothing ("goto" home page, index.php (not logged in of course)) ; 1 : redirected to "access denied" page ; 2 : redirected to an Additional Page precised below. _ the id of the Additional Page which must be used in case of usage of an - unknow code ($ec_conf[4] <=> read_ec_conf('unknown_code_ap_id')). + unknow code ($ec_conf[5] <=> read_ec_conf('unknown_code_ap_id')). *****************************************************************************/ @@ -53,5 +55,5 @@ } else { - $ec_conf = explode(',' , '0,1,10,0,0'); + $ec_conf = explode(',' , '0,1,10,0,0,0'); change_ec_conf('activated', 0); } @@ -80,9 +82,12 @@ $ec_conf[2] = $v; break; - case 'unknown_code': + case 'howto': $ec_conf[3] = $v; break; + case 'unknown_code': + $ec_conf[4] = $v; + break; case 'unknown_code_ap_id': - $ec_conf[4] = $v; + $ec_conf[5] = $v; break; default: return false; @@ -110,6 +115,7 @@ case 'dup_allow' : return $ec_conf[1]; case 'auto_code_dg_nb' : return $ec_conf[2]; - case 'unknown_code' : return $ec_conf[3]; - case 'unknown_code_ap_id': return $ec_conf[4]; + case 'howto' : return $ec_conf[3]; + case 'unknown_code' : return $ec_conf[4]; + case 'unknown_code_ap_id': return $ec_conf[5]; default : return false; } Index: extensions/event_cats/include/evntcats_funcs.inc.php =================================================================== --- extensions/event_cats/include/evntcats_funcs.inc.php (revision 4091) +++ extensions/event_cats/include/evntcats_funcs.inc.php (revision 4092) @@ -121,11 +121,10 @@ * (no return value) */ - function ec_inpect($checked_item, $new_action, - $check_ec_nok = false, $check_forced = false) { + function ec_inpect($checked_item, $new_action, $check_ec_nok = true) { global $ec_lists; $first = array(); $to_correct = array(); - // $to_correct is needed because following code would not work anywhere : + // $to_correct is needed cause following code would not work everywhere : /* foreach ($table as $value) { @@ -154,5 +153,5 @@ if ( $first[$ec_current_code] != $ec_entry[$checked_item] or - ($check_forced and $ec_entry[$checked_item] == 'true') + ($new_action == '' and $ec_entry[$checked_item] == 'true') ) { $to_correct[$ec_current_code] = true; // value not used in fact @@ -166,6 +165,15 @@ foreach ($ec_lists['ec_table'] as &$ec_entry) { // & is needed here if (isset($to_correct[$ec_entry['code']])) { - $ec_entry['action'] = ($check_forced) ? - $ec_entry['action'].$new_action : $new_action; + if ($new_action == '') { + if ( + pwg_query(' + UPDATE `'.EVNTCATS_TABLE.'` + SET `forced` = "false" + WHERE `id` = '.$ec_entry['id'] + ) == null + ) die('Could not fix a "_f_pb"'); + $ec_entry['forced'] = 'false'; + } + else $ec_entry['action'] = $new_action; } } @@ -173,11 +181,11 @@ // Multiple action params for a single code check - ec_inpect('action', 'ec_nok_action_pb', true); + ec_inpect('action', 'ec_nok_action_pb'); // Multiple user_ids for a single code check - ec_inpect('user_id', 'ec_nok_userid_pb'); + ec_inpect('user_id', 'ec_nok_userid_pb', false); // Multiple "forced" params for a single code check - ec_inpect('forced', '_f_pb', true, true); + ec_inpect('forced', ''); // User id and associated page validities checks @@ -308,4 +316,6 @@ */ function ec_create_entry_OK() { + global $page; + $page['errors'][] = 'Y\'a p\'têt\' ben eu une erreur...'; return false; } @@ -322,4 +332,6 @@ */ function ec_duplicate_entry_OK() { + global $page; + $page['errors'][] = 'Y\'a sûrement eu une erreur...'; return false; } @@ -336,4 +348,6 @@ */ function ec_modify_entry_OK() { + global $page; + $page['errors'][] = 'Y\'a pas eu une erreur ?...'; return false; } Index: extensions/event_cats/language/fr_FR/plugin.lang.php =================================================================== --- extensions/event_cats/language/fr_FR/plugin.lang.php (revision 4091) +++ extensions/event_cats/language/fr_FR/plugin.lang.php (revision 4092) @@ -31,8 +31,13 @@ $lang['ec_col_name_dspl'] = 'Page affichée'; $lang['ec_col_name_actn'] = 'Outils'; +$lang['ec_lower_tools_desc'] = 'Entrée sélectionnée :'; $lang['ec_show_URL'] = 'URL : '; +$lang['ec_test_text'] = 'Tester'; +$lang['ec_delete_text'] = 'Supprimer'; +$lang['ec_modify_text'] = 'Modifier'; $lang['ec_url_tooltip'] = 'Copiez l\'adresse du lien pour le transmettre'; $lang['ec_test_tooltip'] = 'Tester l\'URL ( => déconnexion admin.)'; $lang['ec_url_delete'] = 'Supprimer l\'entrée'; +$lang['ec_modify_tooltip'] = 'Modifier l\'entrée'; $lang['ec_url_disable'] = 'Rendre l\'entrée périmée'; $lang['ec_url_forced'] = 'Ne plus forcer ce code à afficher cette page'; @@ -52,5 +57,5 @@ $lang['ec_up_code_lbl_code_old'] = 'Code existant :'; $lang['ec_up_code_lbl_user'] = 'Utilisateur identifié'; -$lang['ec_up_code_lbl_user_none'] = 'Aucun (entrée périmée) :'; +$lang['ec_up_code_lbl_user_none'] = 'Aucun (entrée périmée)'; $lang['ec_up_code_lbl_user_new'] = 'Nouvel utilisateur (créer un compte générique) :'; $lang['ec_new_user_name'] = 'Nom d\'utilisateur'; @@ -67,7 +72,7 @@ // JS checks $lang['ec_confirm_test'] = "Vraiment tester le code sélectionné ?\n\rCela terminera votre session d'administration."; -$lang['ec_confirm_disable'] = "Vraiment rendre périmé le code sélectionné ?\n\rCela supprimera toutes les autres entrées utilisant le même code, s'il y en a."; -$lang['ec_confirm_delete'] = "Vraiment supprimer le code sélectionné ?\n\rCela supprimera toutes les autres entrées utilisant le même code, s'il y en a."; -$lang['ec_confirm_force'] = "Vraiment rendre l'affichage forcé pour le code sélectionné ?\n\rCela supprimera toutes les autres entrées utilisant le même code, s'il y en a."; +$lang['ec_confirm_delete'] = "Vraiment supprimer l'entrée sélectionnée ?"; +$lang['ec_confirm_force'] = "Vraiment forcer l'affichage pour le code sélectionné ?\n\rCela supprimera toutes les autres entrées utilisant le même code, s'il y en a."; +$lang['ec_confirm_del_other'] = "Cela supprimera toutes les autres entrées utilisant le même code, s'il y en a.\n\rOK ?"; $lang['ec_gen_user_act_miss'] = 'Sélectionnez d\'abord une page à afficher'; $lang['ec_act_cat_miss'] = 'Catégorie ?'; @@ -79,4 +84,5 @@ $lang['ec_submit_action_miss'] = 'Vous devez proposer une page à afficher.'; $lang['ec_act_entry_miss'] = 'Sélectionnez d\'abord une entrée à modifier.'; +$lang['ec_clk_entry_miss'] = 'Sélectionnez d\'abord une entrée.'; // In PHP files @@ -84,5 +90,4 @@ $lang['ec_action_pb'] = 'champ "action" corrompu (plusieurs valeurs pour un code ayant plusieurs entrées)'; $lang['ec_username_pb'] = 'champ "user_id" corrompu (plusieurs valeurs pour un code ayant plusieurs entrées)'; -$lang['ec_forced_pb'] = 'champ "forced" corrompu (au moins une fois "true" pour un code ayant plusieurs entrées)'; $lang['ec_username_miss'] = 'utilisateur inexistant (probablement supprimé)'; $lang['ec_category_miss'] = 'catégorie inexistante (probablement supprimée)'; @@ -110,11 +115,7 @@ // actions $lang['ec_ok'] = 'OUI'; -$lang['ec_ok_f_pb'] = 'OUI (page non forcée)'; $lang['ec_ok_ap_pb'] = 'OUI (Add. Page inconnue)'; $lang['ec_ok_cat_pb'] = 'OUI (catégorie inconnue)'; $lang['ec_ok_img_pb'] = 'OUI (image inconnue)'; -$lang['ec_ok_f_pb_ap_pb'] = 'OUI (page non forcée et Add. Page inconnue)'; -$lang['ec_ok_f_pb_cat_pb'] = 'OUI (page non forcée et catégorie inconnue)'; -$lang['ec_ok_f_pb_img_pb'] = 'OUI (page non forcée et image inconnue)'; $lang['ec_nok'] = 'NON'; $lang['ec_nok_ap_pb'] = 'NON'; Index: extensions/event_cats/main.inc.php =================================================================== --- extensions/event_cats/main.inc.php (revision 4091) +++ extensions/event_cats/main.inc.php (revision 4092) @@ -122,8 +122,5 @@ if (isset($_GET['cat'])) $ec_cat = $_GET['cat']; if (isset($_GET['img'])) $ec_img = $_GET['img']; - if ( - $ec_entry['forced'] == 'true' and - !is_in($ec_entry['action'], '_f_pb') - ) { + if ($ec_entry['forced'] == 'true') { if (empty($ec_entry['arg1']) and !empty($ec_entry['arg2'])) { $ec_ap = $ec_entry['arg2']; Index: extensions/event_cats/maintain.inc.php =================================================================== --- extensions/event_cats/maintain.inc.php (revision 4091) +++ extensions/event_cats/maintain.inc.php (revision 4092) @@ -110,16 +110,7 @@ the user ids table ; ec_nok_ap_pb : not valid Additional Page id -> access denied ; - ec_ok_f_pb : the same "code" is used in more than one entries in the - DB (which is allowed) and the "forced" argument is - "true" for at least one of these entries, which should - not be possible, as a "forced" code can occur only once - in the DB. User is logged in and the code behaves as a - not "forced" code ; ec_ok_ap_pb : not valid Additional Page id -> home page ; ec_ok_cat_pb : not valid category id -> home page ; ec_ok_img_pb : not valid image id -> category page ; - ec_ok_f_pb_ap_pb : ec_ok_ap_pb + "forced problem" -> home page ; - ec_ok_f_pb_cat_pb : ec_ok_cat_pb + "forced problem" -> home page ; - ec_ok_f_pb_img_pb : ec_ok_img_pb + "forced problem" -> category page ; _ the two main fields are "user_id" and "code", as the main purpose of the |