php dom load form and create element inside

Question

What im trying to do is load a form and create elements inside it but i end up with something like this:

<form id="form_54" accept-charset="utf-8"></form><input type="text" name="name">

The outpot i'm lookin for is:

<form id="form_54" accept-charset="utf-8"><input type="text" name="name"></form>

this is my function:

public function input($name, $attributes = array(), $type = 'text')
{
    $form = new DOMDocument();
    $form->loadXML($this->doc->saveHTML());

    $input = $form->createElement('input');
    $input->setAttribute('type', $type);
    $input->setAttribute('name', $name);

    if(isset($attributes))
    {
        foreach($attributes as $attr => $val)
        {
            $input->setAttribute($attr, $val);
        }
    }

    $form->appendChild($input);
    $this->doc->loadXML($form->saveHTML());
}

Correct function thanks to Ghost:

public function input($name, $attributes = array(), $type = 'text')
{
    $form = $this->doc->getElementsByTagName('form')->item(0);

    $input = $this->doc->createElement('input');
    $input->setAttribute('type', $type);
    $input->setAttribute('name', $name);

    if(isset($attributes))
    {
        foreach($attributes as $attr => $val)
        {
            $input->setAttribute($attr, $val);
        }
    }

    $form->appendChild($input);
    $this->doc->appendChild($form);
}

so whats the problem?

Kevin
– Kevin

2015-04-28 12:51:11 +00:00
Commented Apr 28, 2015 at 12:51 — Kevin
– Kevin, Commented Apr 28, 2015 at 12:51
Where are you adding form? Is it already in DOM?

Manwal
– Manwal

2015-04-28 12:57:15 +00:00
Commented Apr 28, 2015 at 12:57 — Manwal
– Manwal, Commented Apr 28, 2015 at 12:57

Kevin · Accepted Answer · 2015-04-28 12:57:55Z

1

Most likely you're appending on the parent element instead of the form. Try to target the form first, then making the append.

public function input($name, $attributes = array(), $type = 'text')
{
    $dom = new DOMDocument();
    $dom->loadXML($this->doc->saveHTML());
    // target the form
    $form = $dom->getElementsByTagName('form')->item(0);

    $input = $dom->createElement('input');
    $input->setAttribute('type', $type);
    $input->setAttribute('name', $name);

    if(isset($attributes))
    {
        foreach($attributes as $attr => $val)
        {
            $input->setAttribute($attr, $val);
        }
    }

    $form->appendChild($input);
    $this->doc->loadXML($form->saveHTML());
}

answered Apr 28, 2015 at 12:57

Kevin

41.9k12 gold badges57 silver badges72 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

jailbird.phoenix Over a year ago

your solution didn't work. however the getElementsByTagName('form')->item(0); that you mentioned was what i needed. so i ended up with the correct function with i added to my question. Thank You!

Kevin Over a year ago

@jailbird.phoenix oh yeah, i forgot the instance of of your object was inside ->doc, anyway glad this helped

Collectives™ on Stack Overflow

php dom load form and create element inside

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related