Vinnaren i pepparkakshustävlingen!
2012-01-27, 09:04
  #1
Medlem
Jag suttit ett tag nu och försökt göra så att admin granskar inlägg innan de accepteras, men jag får inte till det.
Just nu får jag mysql error: "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ållet, 0, 36)' at line 2"

Jag har kollat i databasen om kolumnerna stämmer, men jag hittar verkligen inte vad som går snett.

Här är koden:
Kod:
echo '<h1>Ogranskade inlägg</h1>';

$sql_unconfirmed_posts "SELECT * FROM unconfirmed_posts ORDER BY post_time DESC";
$res_unconfirmed_posts mysql_query($sql_unconfirmed_posts) or die (mysql_error());

if(isset(
$_SESSION['uid']) && $_SESSION['user_level'] == 1){
    
    if(
mysql_num_rows($res_unconfirmed_posts) > 0){
        
        while(
$row mysql_fetch_assoc($res_unconfirmed_posts)){
         
         echo 
"
            <div id='checkup_posts_div'><h1>Ogranskade inlägg</h1><form method='GET' action=''>
            
            <div id='checkup_post_title'><h1>Rubrik:<textarea name='unconf_post_title'>" 
.$row['post_title'] . "</textarea></h1></div>
            
            <div id='checkup_post_desc'><b>Kort inledning 510 tecken:</b><textarea name='unconf_post_desc'>" 
.$row['post_desc']. "</textarea></div>
            
            <div id='checkup_post_content'><p>Innehåll 5000 tecken:</p><textarea name='unconf_post_content'>" 
$row['post_content'] ."</textarea></div>
            
            <b>Post rank:</b><input type='text' name='unconf_post_rank' value="
.$row['post_rank']."><br>
            
            <p>post ID:"
.$row['post_id']."</p><input type='hidden' name='unconf_post_id' value=".$row['post_id']." /><br>
            
            <input type='submit' name='send_unconf_to_conf' value='Acceptera inlägg'><input type='submit' name='send_unconf_to_denied' value='Neka inlägg'>
            
            </form></div>"
;
    
            }
            if(isset(
$_GET['send_unconf_to_conf'])){
                
                echo 
$_GET['unconf_post_id']; 
                
                
$sql_unconf_to_conf "INSERT INTO posts(post_title, post_desc, post_content, post_rank, post_id) 
                VALUES("
.$_GET['unconf_post_title'].", ".$_GET['unconf_post_desc'].", ".$_GET['unconf_post_content'].", ".$_GET['unconf_post_rank'].", ".$_GET['unconf_post_id'].") 
                "
;
                
                
mysql_query($sql_unconf_to_conf) or die (mysql_error());
            }
    }
    else{
        echo 
'Det finns inga ogranskade inlägg';
    }

Citera
2012-01-27, 12:49
  #2
Moderator
Protons avatar
Har du läst tråden https://www.flashback.org/t1719347? Borde ju kunna hjälpa dej att hitta felet?

För övrigt förstår jag inte varför du ska göra det så svårt för dej med att flytta data. när du ska ha inlägg som inte ska synas men som du vill kunna visa upp senarfe.

Varför inte införa ett extra attribut i tabellen som heter "publicerat" eller nåt sånt och lägga in en 0 för ej publicerat och 1 för publicerat? På det sättet kommer du ju undan med en uppdatering av tabellen istället för att flytta data?
__________________
Senast redigerad av Proton 2012-01-27 kl. 12:52.
Citera
2012-01-27, 13:04
  #3
Medlem
Citat:
Ursprungligen postat av Proton
Har du läst tråden https://www.flashback.org/t1719347? Borde ju kunna hjälpa dej att hitta felet?

För övrigt förstår jag inte varför du ska göra det så svårt för dej med att flytta data. när du ska ha inlägg som inte ska synas men som du vill kunna visa upp senarfe.

Varför inte införa ett extra attribut i tabellen som heter "publicerat" eller nåt sånt och lägga in en 0 för ej publicerat och 1 för publicerat? På det sättet kommer du ju undan med en uppdatering av tabellen istället för att flytta data?
Felet var att all GET-data som jag satte in inte var omsluten med ' .

Det kanske är lite väl onödigt att skapa två olika tabeller för dessa, men jag har gjort så för att vara långsiktig då inläggen ska behandlas lite olika.
Bl.a ska vissa som är äldre än 1 vecka tas bort och liknande, detta går såklart också bra att göra med en ny kolumn som du säger.

Men det kändes "säkrast" att göra på det här viset.
Citera

Stöd Flashback

Flashback finansieras genom donationer från våra medlemmar och besökare. Det är med hjälp av dig vi kan fortsätta erbjuda en fri samhällsdebatt. Tack för ditt stöd!

Stöd Flashback