2013-06-11 2 views
0

Это более чем вероятно то, что я просто глупо, но я не могу найти, где мой код, кажется, идет не так. Я пытаюсь подготовить команду SQL UPDATE для страницы редактирования для внутренней (да, я знаю, что я не кодирую в текущей версии PHP), работа требует быстрого исправления и ее внутренней системы - пожалуйста, не судите меня !) и независимо от того, что я, кажется, делаю, я постоянно сталкиваюсь с ошибкой, когда мне говорят, что у меня есть дополнительная; в моем заявлении. Однако команда UPDATE, которую я собираюсь опубликовать, почти идентична (за исключением нескольких полей) другой, которую я подготовил для почти идентичной части базы данных в другом месте, и это работает отлично! Я чувствую, что я просто пропускаю какую-то пунктуацию где-то, но я смотрел на нее часами, поэтому я надеюсь, что кто-то еще увидит. Я на самом деле разместить весь блок кода, а не только заявления UPDATE, на всякий случай ....Нужна помощь в поиске ошибки кодирования

<?php 
require("header.php"); 
$error_feat_id = ""; 
$valid_feature = ""; 
$error_feat_id = ""; 
$refferal_feat_id = $_GET['feat_id']; 
if(isset($refferal_feat_id) == TRUE) { 
    if(is_numeric($refferal_feat_id) == FALSE) { 
     $error_feat_id = 1; 
    } 
    if($error_feat_id == 1) { 
     header("Location: ".$config_basedir); 
    } 
    else { 
     $valid_feature = $refferal_feat_id; 
    } 
} 
else { 
    $valid_feature = 0; 
} 

if($_POST['submit']) { 
    $datereceived = $_POST['calendaryear']."-".$_POST['calendarmonth']."-".$_POST['calendarday']; 
    $director1dob = $_POST['director1year']."-".$_POST['director1month']."-".$_POST['director1day']; 
    $director2dob = $_POST['director2year']."-".$_POST['director2month']."-".$_POST['director2day']; 
    $update_feature_sql = "UPDATE features SET Title = '".mysql_real_escape_string($_POST['title'])."', 
    DateReceived = STR_TO_DATE(".$datereceived.", '%Y-%m-%d'), 
    Synopsis = '".mysql_real_escape_string($_POST['synopsis'])."', 
    LogLine = '".mysql_real_escape_string($_POST['logline']."', 
    StrandID = ".$_POST['programstrand'].", 
    CompCatID = ".$_POST['compcat'].", 
    PremiereID = ".$_POST['premierestatus'].", 
    FirstFeatureID = ".$_POST['firstfeature'].", 
    OriginID = ".$_POST['origin'].", 
    RoughtCutID = ".$_POST['roughcut'].", 
    ScreenerID = ".$_POST['screener'].", 
    Director1 = '".mysql_real_escape_string($_POST['director1'])."', 
    Director1Bio = '".mysql_real_escape_string($_POST['director1bio'])."', 
    Director1Nationality = '".mysql_real_escape_string($_POST['director1nation'])."', 
    Director1DOB = STR_TO_DATE(".$director1dob.", '%Y-%m-%d'), 
    Director2 = '".mysql_real_escape_string($_POST['director2'])."', 
    Director2Bio = '".mysql_real_escape_string($_POST['director2bio'])."', 
    Director2Nationality = '".mysql_real_escape_string($_POST['director2nation'])."', 
    Director2DOB = STR_TO_DATE(".$director2dob.", '%Y-%m-%d'), 
    Country1 = '".mysql_real_escape_string($_POST['country1'])."', 
    Country2 = '".mysql_real_escape_string($_POST['country2'])."', 
    Country3 = '".mysql_real_escape_string($_POST['country3'])."', 
    Country4 = '".mysql_real_escape_string($_POST['country4'])."', 
    Country5 = '".mysql_real_escape_string($_POST['country5'])."', 
    Country6 = '".mysql_real_escape_string($_POST['country6'])."', 
    Country7 = '".mysql_real_escape_string($_POST['country7'])."', 
    Country8 = '".mysql_real_escape_string($_POST['country8'])."', 
    Country9 = '".mysql_real_escape_string($_POST['country9'])."', 
    Country10 = '".mysql_real_escape_string($_POST['country10'])."', 
    Cast1 = '".mysql_real_escape_string($_POST['cast1'])."', 
    Cast2 = '".mysql_real_escape_string($_POST['cast2'])."', 
    Cast3 = '".mysql_real_escape_string($_POST['cast3'])."', 
    Cast4 = '".mysql_real_escape_string($_POST['cast4'])."', 
    AdditionalComments = '".mysql_real_escape_string($_POST['additionalcomments'])."', 
    DurationHours = '".mysql_real_escape_string($_POST['durationhours'])."', 
    DurationMins = '".mysql_real_escape_string($_POST['durationmins'])."', 
    ScreeningFormat1 = ".$_POST['screeningformat1'].", 
    ScreeningFormat2 = ".$_POST['screeningformat2'].", 
    ScreeningFormat3 = ".$_POST['screeningformat3'].", 
    ScreeningFormat4 = ".$_POST['screeningformat4'].", 
    ScreeningFormat5 = ".$_POST['screeningformat5'].", 
    DCPEID = ".$_POST['dcpe'].", 
    DCPCPL = '".mysql_real_escape_string($_POST['dcpcpl'])."', 
    SubTypeID = ".$_POST['subtype'].", 
    AspectID = ".$_POST['aspect'].", 
    LanguageID = ".$_POST['language'].", 
    SubtitlesID = ".$_POST['subtitles'].", 
    AudioID = ".$_POST['audio'].", 
    PubRecID = ".$_POST['pubrec'].", 
    TypeOfMaterials = '".mysql_real_escape_string($_POST['typeofmaterials'])."', 
    ContactName = '".mysql_real_escape_string($_POST['contactname'])."', 
    ContactEmail = '".mysql_real_escape_string($_POST['contactemail'])."', 
    ContactPhone1 = '".mysql_real_escape_string($_POST['contactphone1'])."', 
    ContactPhone2 = '".mysql_real_escape_string($_POST['contactphone2'])."', 
    ContactAddress = '".mysql_real_escape_string($_POST['contactaddress'])."', 
    HomeAddressID = ".$_POST['homeaddress'].", 
    ContactWebsite = '".mysql_real_escape_string($_POST['contactwebsite'])."', 
    SAERecID = ".$_POST['saerec'].", 
    SAERetID = ".$_POST['saeret'].", 
    FeeStatusID = ".$_POST['feestatus'].", 
    EntryFeeID = ".$_POST['entryfee'].", 
    ProdYearID = '".mysql_real_escape_string($_POST['prodyear'])."', 
    ProdCo1 = '".mysql_real_escape_string($_POST['prodco1'])."', 
    ProdWebsite1 = '".mysql_real_escape_string($_POST['prodwebsite1'])."', 
    ProdCo1Nation = '".mysql_real_escape_string($_POST['prodco1nation'])."', 
    ProdCo2 = '".mysql_real_escape_string($_POST['prodco2'])."', 
    ProdWebsite2 = '".mysql_real_escape_string($_POST['prodwebsite2'])."', 
    ProdCo2Nation = '".mysql_real_escape_string($_POST['prodco2nation'])."', 
    Producer1 = '".mysql_real_escape_string($_POST['producer1'])."', 
    Producer1Nation = '".mysql_real_escape_string($_POST['producer1nation'])."', 
    Producer2 = '".mysql_real_escape_string($_POST['producer2'])."', 
    Producer2Nation = '".mysql_real_escape_string($_POST['producer2nation'])."', 
    Producer3 = '".mysql_real_escape_string($_POST['producer3'])."', 
    Producer3Nation = '".mysql_real_escape_string($_POST['producer3nation'])."', 
    Producer4 = '".mysql_real_escape_string($_POST['producer4'])."', 
    Producer4Nation = '".mysql_real_escape_string($_POST['producer4nation'])."', 
    SalesAgent = '".mysql_real_escape_string($_POST['salesagent'])."', 
    Distributor = '".mysql_real_escape_string($_POST['distributor'])."', 
    TalentAttached = '".mysql_real_escape_string($_POST['talentattached'])."', 
    PreviouslyScreenedAt = '".mysql_real_escape_string($_POST['prevscreenat'])."', 
    ScreeningFeesID = ".$_POST['screeningfees'].", 
    ScreeningFeesAmt = '".mysql_real_escape_string($_POST['screeningfeesamt'])."', 
    KidsID = ".$_POST['kids'].", 

код показывает ошибку на этой линии

ConfirmID = ".$_POST['confirm']." WHERE ID = ".$refferral_feat_id.";"; 


mysql_query($update_feature_sql) or die(mysql_error()); 
header("Location: ".$config_basedir."feat_show.php?feat_id=".$refferal_feat_id); 
} 
else { 
    $sql_get_feature = "SELECT * FROM features WHERE ID = ".$refferal_feat_id.";"; 
    $query_get_feature = mysql_query($sql_get_feature); 
    $rows_get_feature = mysql_fetch_assoc($query_get_feature); 
?> 

Я понимаю, что я вероятно, что-то упустил из виду, и что я мог бы делать что-то более эффективно, но я PHP noob, который сразу же поднимает стены с несколькими вещами, и может действительно сделать с этим кодом! Заранее благодарим за помощь/совет!

+1

Подсветка синтаксиса наводит на размышления, не так ли? :) –

+0

Какова ценность ConfirmID, которая должна быть? – JonnyS

+0

Вы получаете синтаксическую ошибку PHP или SQL-версию? –

ответ

1

Эта первая строка в вашем втором блоке кода имеет 5 двойных кавычек. Строка не завершена.

+1

Я предполагаю, что это продолжение первого блока, с комментарием ФП, заполненным посередине. –

0

ну, похоже, у вас есть дополнительный; ты пробовал?

ConfirmID = $_POST['confirm']." WHERE ID = ".$refferral_feat_id; 
0

это кажется неправильным

ConfirmID = ".$_POST['confirm']." WHERE ID = ".$refferral_feat_id.";"; 

попытка изменить его в:

ConfirmID = $_POST['confirm']." WHERE ID = ".$refferral_feat_id; 
+0

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

0

Хорошо, принимая во внимание, что ConfirmID является частью строки определяется над ним.

.$_POST['kids'].", ConfirmID = ".$_POST['confirm']." WHERE ID = ".$refferral_feat_id; 

Я считаю, что нижняя строка должна работать. Запросы SQL, переданные в mysql_query(), не заканчиваются точкой с запятой.

ConfirmID = ".$_POST['confirm']." WHERE ID = ".$refferral_feat_id; 
Смежные вопросы