why image unlink works when condition false












0















I have a form that can either add icon or a icontext. The problem is, that icon is unlinked from file path when I submit form without change.



enter image description here



if (isset($_POST['icontext'])) { // echo '<pre>';print_r($_POST['icontext']);exit;  
if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) {
unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);
//echo '<pre>';print_r($this->view->highlights['highlighticon']);exit;
}
}









share|improve this question

























  • maybe you want !empty($_POST['icontext']) instead of isset? icontext could/will be set even if it's an empty string.

    – Jeff
    Jan 1 at 13:45













  • You might want to consider accepting an answer if it helped you.

    – Jeff
    Jan 1 at 14:01
















0















I have a form that can either add icon or a icontext. The problem is, that icon is unlinked from file path when I submit form without change.



enter image description here



if (isset($_POST['icontext'])) { // echo '<pre>';print_r($_POST['icontext']);exit;  
if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) {
unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);
//echo '<pre>';print_r($this->view->highlights['highlighticon']);exit;
}
}









share|improve this question

























  • maybe you want !empty($_POST['icontext']) instead of isset? icontext could/will be set even if it's an empty string.

    – Jeff
    Jan 1 at 13:45













  • You might want to consider accepting an answer if it helped you.

    – Jeff
    Jan 1 at 14:01














0












0








0








I have a form that can either add icon or a icontext. The problem is, that icon is unlinked from file path when I submit form without change.



enter image description here



if (isset($_POST['icontext'])) { // echo '<pre>';print_r($_POST['icontext']);exit;  
if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) {
unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);
//echo '<pre>';print_r($this->view->highlights['highlighticon']);exit;
}
}









share|improve this question
















I have a form that can either add icon or a icontext. The problem is, that icon is unlinked from file path when I submit form without change.



enter image description here



if (isset($_POST['icontext'])) { // echo '<pre>';print_r($_POST['icontext']);exit;  
if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) {
unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);
//echo '<pre>';print_r($this->view->highlights['highlighticon']);exit;
}
}






php html






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Jan 1 at 13:49









Sebastian Brosch

23.8k124153




23.8k124153










asked Jan 1 at 13:37









HemilHemil

315




315













  • maybe you want !empty($_POST['icontext']) instead of isset? icontext could/will be set even if it's an empty string.

    – Jeff
    Jan 1 at 13:45













  • You might want to consider accepting an answer if it helped you.

    – Jeff
    Jan 1 at 14:01



















  • maybe you want !empty($_POST['icontext']) instead of isset? icontext could/will be set even if it's an empty string.

    – Jeff
    Jan 1 at 13:45













  • You might want to consider accepting an answer if it helped you.

    – Jeff
    Jan 1 at 14:01

















maybe you want !empty($_POST['icontext']) instead of isset? icontext could/will be set even if it's an empty string.

– Jeff
Jan 1 at 13:45







maybe you want !empty($_POST['icontext']) instead of isset? icontext could/will be set even if it's an empty string.

– Jeff
Jan 1 at 13:45















You might want to consider accepting an answer if it helped you.

– Jeff
Jan 1 at 14:01





You might want to consider accepting an answer if it helped you.

– Jeff
Jan 1 at 14:01












2 Answers
2






active

oldest

votes


















0














I think you want to check whether $_POST['icontext'] is empty or not, instead of checking whether it is_set or not, because it might be set even if it's only an empty string.



So change your code to



if (!empty($_POST['icontext'])) {
//...
}





share|improve this answer
























  • yaa, its working now

    – Hemil
    Jan 1 at 13:51



















0














if (!empty($_POST['icontext'])) {

if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) { unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);



  }
}





share|improve this answer























    Your Answer






    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "1"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53995902%2fwhy-image-unlink-works-when-condition-false%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown

























    2 Answers
    2






    active

    oldest

    votes








    2 Answers
    2






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    0














    I think you want to check whether $_POST['icontext'] is empty or not, instead of checking whether it is_set or not, because it might be set even if it's only an empty string.



    So change your code to



    if (!empty($_POST['icontext'])) {
    //...
    }





    share|improve this answer
























    • yaa, its working now

      – Hemil
      Jan 1 at 13:51
















    0














    I think you want to check whether $_POST['icontext'] is empty or not, instead of checking whether it is_set or not, because it might be set even if it's only an empty string.



    So change your code to



    if (!empty($_POST['icontext'])) {
    //...
    }





    share|improve this answer
























    • yaa, its working now

      – Hemil
      Jan 1 at 13:51














    0












    0








    0







    I think you want to check whether $_POST['icontext'] is empty or not, instead of checking whether it is_set or not, because it might be set even if it's only an empty string.



    So change your code to



    if (!empty($_POST['icontext'])) {
    //...
    }





    share|improve this answer













    I think you want to check whether $_POST['icontext'] is empty or not, instead of checking whether it is_set or not, because it might be set even if it's only an empty string.



    So change your code to



    if (!empty($_POST['icontext'])) {
    //...
    }






    share|improve this answer












    share|improve this answer



    share|improve this answer










    answered Jan 1 at 13:50









    JeffJeff

    6,34911025




    6,34911025













    • yaa, its working now

      – Hemil
      Jan 1 at 13:51



















    • yaa, its working now

      – Hemil
      Jan 1 at 13:51

















    yaa, its working now

    – Hemil
    Jan 1 at 13:51





    yaa, its working now

    – Hemil
    Jan 1 at 13:51













    0














    if (!empty($_POST['icontext'])) {

    if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) { unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);



      }
    }





    share|improve this answer




























      0














      if (!empty($_POST['icontext'])) {

      if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) { unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);



        }
      }





      share|improve this answer


























        0












        0








        0







        if (!empty($_POST['icontext'])) {

        if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) { unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);



          }
        }





        share|improve this answer













        if (!empty($_POST['icontext'])) {

        if(file_exists(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon'] )) { unlink(UPLOAD_PATH . 'images/highlights_icon/' . $this->view->highlights['highlighticon']);



          }
        }






        share|improve this answer












        share|improve this answer



        share|improve this answer










        answered Jan 1 at 13:50









        HemilHemil

        315




        315






























            draft saved

            draft discarded




















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53995902%2fwhy-image-unlink-works-when-condition-false%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Monofisismo

            Angular Downloading a file using contenturl with Basic Authentication

            Olmecas