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

Fixed nokogiri::fromDom($dom) #9

Closed
wants to merge 5 commits into from
Closed

Fixed nokogiri::fromDom($dom) #9

wants to merge 5 commits into from

Conversation

positronium
Copy link
Contributor

При передаче DOMElement в nokogiri::fromDom(), а затем
вызове $nokogiri->get(' ... ') падает, т.к. метод getDom() возвращает пустоту.

Stack trace:
#0 nokogiri.php(132): DOMXPath->__construct(NULL)
#1 nokogiri.php(198): nokogiri->getXpath()
#2 nokogiri.php(109): nokogiri->getElements('//*[contains(co...')
#3 index.php(70): nokogiri->get('.some_class...')
#4 {main}  thrown in nokogiri.php on line 132```

positronium and others added 3 commits December 17, 2013 17:00
При передаче DOMElement в nokogiri::fromDom(), а затем
 вызове $nokogiri->get(' ... ') падает, т.к. метод getDom() возвращает пустоту.
Fatal error:  Uncaught exception 'DOMException' with message 'DOMXPath::__construct() expects parameter 1 to be DOMDocument, null given' in nokogiri.php:132
Stack trace:
#0 nokogiri.php(132): DOMXPath->__construct(NULL)
#1 nokogiri.php(198): nokogiri->getXpath()
#2 nokogiri.php(109): nokogiri->getElements('//*[contains(co...')
#3 index.php(70): nokogiri->get('.some_class...')
#4 {main}  thrown in nokogiri.php on line 132
@olamedia
Copy link
Owner

Сделал вручную, с некоторыми изменениями. Если можно, неплохо бы проверить, ибо правил прямо на гитхабе и не тестил. может не работать.

@olamedia
Copy link
Owner

Можно уточнить какая цель у toTextArray() и toText() в вашем случае?

@positronium
Copy link
Contributor Author

toText() должен был возвращать текстовое представление всей выборки, как .text() в JQuery.
toTextArray() - массив таких представлений для каждого элемента.
Просто результатом выборки может оказаться <div><span>Text</span></div> вместо <div>Text</div>
Сделал для таких случаев. И не думал, что эти коммиты попадут в пулл реквест. Почитаю документацию гитхаба в праздники.

olamedia added a commit that referenced this pull request Dec 18, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants