Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New operator ! for innerText #15

Closed
wants to merge 4 commits into from
Closed

New operator ! for innerText #15

wants to merge 4 commits into from

Conversation

Finnepinnen
Copy link

The ! operator just return everything that is in the active element

The parser
'title' => ['uses' => 'title[::lang>locale,!>name]'],

Applied on
<?xml version="1.0" encoding="UTF-8"?> <course code="ABC"> <title lang="sv">Utmattning</title> <title lang="en">Fatigue</title> </course>

Returns
`Array
(
[code] => ABC
[title] => Array
(
[0] => Array
(
[locale] => sv
[name] => Utmattning
)

        [1] => Array
            (
                [locale] => en
                [name] => Fatigue
            )
    )

)`

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.7%) to 89.82% when pulling 949b9e2 on Finnepinnen:master into 168b081 on laravie:master.

@crynobone
Copy link
Member

Please resend PR with tests.

@crynobone crynobone closed this Apr 15, 2018
@crynobone
Copy link
Member

In fact this is not needed.

See 0f0cd8f and the tests return expected value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants