"маленькие" удобства - вопрос новичка

[Gibiskus]

хочется побороть одну не большую проблему:

function paste_in_textarea() {
    function edit_content() { //cоздать ссылки на страницы которые уже есть в базе 
        $tpl=""; 
        $sql = mysql_query('SELECT * FROM content');// выводим поля, ссылки и чекбокс
        while($record = mysql_fetch_array($sql)) {
     
    
            $tpl .= '<tr><td>'.$record['pageid'].': '.$record['pagetittle'].'</td><td>
            <a href="'.$_SERVER['PHP_SELF'].'?action=paste_in_textarea&pageid='.$record['pageid'].'"><img src="collapsed.gif"/></a>
            <a href="'.$_SERVER['PHP_SELF'].'?action=paste_in_textarea&pagedel='.$record['pageid'].'&delete=y"><img src="close.png"/></a>
           </td>';
        }
    return $tpl;
    }//edit_content()
    $html = '';
    if(isset($_GET['pageid'])) {
        $sql = mysql_query("select * FROM content WHERE pageid='".$_GET['pageid']."';"); //вставка контента в текстовую форму
        while($record = mysql_fetch_array($sql)) {
        $content = $record['pagecontent'];
        $title = $record['pagetittle'];
        $discription = $record['pagediscription'];
        $keywords = $record['pagekeywords'];
        }
    }
    else {
        $content="";
        $title ="";
        $discription ="";
        $keywords ="";
        $_GET['pageid']=0;
    }

    if(isset($_GET['pagedel'])) { //кнопка удалить
        if($_GET['pagedel']) {
        $sql = mysql_query ("DELETE FROM content WHERE pageid='".$_SESSION['id']."';");  
        header('Refresh:0; URL=index.php?action=paste_in_textarea'); 
        }
    }

    if(isset($_POST['save'])) { //кнопка сохранить
        if($_POST['save']) { 
        $sql= mysql_query ("UPDATE content SET pagecontent = '".$_POST['mess']."', pagetittle = '".$_POST['tittle']."',
        pagediscription = '".$_POST['discription']."', pagekeywords = '".$_POST['keywords']."', cheked = '".$_POST['chek']."' WHERE pageid='".$_SESSION['id']."';");
        } 
    }
    $_SESSION['id'] = $_GET['pageid'];  // id записи = ид сессии
    
   
//,,,,,,,
}//paste_in_textarea()

Каждый раз когда будет нужно провести какие то либо действия нужно нажать иконку "редактировать"...
например если нужно "удалить", то придется нажать на иконку "редактировать", а только потом на иконку "удалить", то же самое и с чекбоксом, если хочу проставить или убрать нужно жать редактировать, потом кнопку "сохранить" и только тогда...
Как освободится от этих "пут"?

 

[KorP]

Как освободится от этих "пут"?

реализовать по другому? ))
что мешает pagedel вынести в отдельную функцию?

 

[Gibiskus]

у меня получилось как то так, без сессии

    if(isset($_GET['pagedel'])) {
        $sql = mysql_query ("DELETE FROM content WHERE pageid='".$_GET['pagedel']."';");  
    }

что мешает pagedel вынести в отдельную функцию?

скажите пожалуйста зачем выносить в отдельную?

 

[KorP]

по тому что честно сказать у меня волосы на одном месте шевелятся когда я вижу данный код
одна функция в другой, хтмл там намешан, передаваемые данные не проверяются
на мой взгляд в данном случае

if(isset($_GET['pagedel'])) {...}

вообще работать не должно так как нужно
Refresh:0? а Location?

зачем выносить в отдельную?

наверное по тому что ты написал вызывает у тебя же проблемы с его использованием

я бы всё это удалил быстренько и написал с нуля

 

[Gibiskus]

я бы всё это удалил быстренько и написал с нуля

спс буду, но мне рано пока что все менять , необходимо еще некоторые моменты понять и разобрать проблемы, для того что бы наладить самочувствие, перестать беспокоится и начать жить))

 

[KorP]

все менять

ну да, из одной функции сделать 3 это конечно проблема, целых пару минут копипаста

 

[Gibiskus]

все менять

это наверно было о "глобальном"

целых пару минут копипаста

таки да - не такая большая проблема