Activity for PHP Simple HTML DOM Parser

Hi Man O ManaO modified a comment on ticket #63

Hi Man O ManaO — Mon, 02 Dec 2024 02:52:53 -0000

Ok. Wrong test. The method it's called like $sections = $html->find('section')->firstChild(); but I got the same an error because it's an Array. So there's not the same as the CSS pseudo selector rule :first-child How to use it to get the same result as CSS? TNX

Hi Man O ManaO modified a comment on ticket #63

Hi Man O ManaO — Mon, 02 Dec 2024 02:51:40 -0000

Hi Man O ManaO modified a comment on ticket #63

Hi Man O ManaO — Mon, 02 Dec 2024 02:51:26 -0000

Hi Man O ManaO posted a comment on ticket #63

Hi Man O ManaO — Mon, 02 Dec 2024 02:50:52 -0000

Hi Man O ManaO posted a comment on ticket #210

Hi Man O ManaO — Sat, 30 Nov 2024 21:30:48 -0000

Sorry, wrong goal. Close this. The correct answer it's here: https://sourceforge.net/p/simplehtmldom/support-requests/63/

Hi Man O ManaO created ticket #63

Hi Man O ManaO — Sat, 30 Nov 2024 20:52:01 -0000

Get the elements of the upper level like with CSS pseudo selector :first-child

Hi Man O ManaO created ticket #210

Hi Man O ManaO — Sat, 30 Nov 2024 10:10:25 -0000

Find first child element like CSS does not respect order

Maxim Volobuev created ticket #209

Maxim Volobuev — Thu, 21 Nov 2024 17:58:00 -0000

Decoding HTML entities corrupts text in HTML

Igor Zhuravlov posted a comment on ticket #208

Igor Zhuravlov — Tue, 17 Oct 2023 05:09:46 -0000

This bug persists even with well-formed HTML with single root element:

EOT; ...

Igor Zhuravlov created ticket #208

Igor Zhuravlov — Tue, 17 Oct 2023 01:35:04 -0000

$node->find() finds element next to $node

Kowsar Hossain posted a comment on ticket #207

Kowsar Hossain — Tue, 07 Mar 2023 07:20:38 -0000

I've also submitted a patch to address this issue here: https://sourceforge.net/p/simplehtmldom/feature-requests/68/

Kowsar Hossain created ticket #68

Kowsar Hossain — Tue, 07 Mar 2023 07:18:57 -0000

Added feature to enable/disable htmlentity operations

Kowsar Hossain created ticket #207

Kowsar Hossain — Tue, 07 Mar 2023 06:55:57 -0000

The output doesn't match the input even when the input hasn't been modified

DennisKmetz created ticket #62

DennisKmetz — Thu, 23 Feb 2023 16:05:05 -0000

PHP 8.x support

olalav created ticket #67

olalav — Mon, 06 Feb 2023 09:59:05 -0000

Get only text in leaf nodes (avoid duplication)

hkirsman created ticket #61

hkirsman — Thu, 19 Jan 2023 19:30:08 -0000

Is Github page safe to use for downloads?

olalav created ticket #205

olalav — Wed, 11 Jan 2023 07:44:34 -0000

End tags erroneously included in plaintext

Philip posted a comment on a blog post

Philip — Wed, 04 Jan 2023 15:25:35 -0000

There is a typo in the command, missing the "p" in the second "simple." shoulde be composer require simplehtmldom/simplehtmldom dev-master

Coz posted a comment on ticket #185

Coz — Wed, 21 Dec 2022 02:37:08 -0000

This is not a bug in simplehtmldom Yes it is. You're not setting a user agent in either the curl code or the stream_context code. Any properly configured server will reject the requests, which makes the project useless. You need to either add a generic user agent (recommend google bot) or provide a way for the user to pass in their own user agent to the function. See lines 72 and 111 of revised HtmlWeb.php file.

olalav posted a comment on ticket #203

olalav — Fri, 09 Dec 2022 10:13:39 -0000

Actually it has to be done something like this, because this function can be called from inside the library, and we want to get the first call that is outside the library. PS! Is the maintainer active these days? Has been quiet for a while. diff --git a/HtmlNode.php b/HtmlNode.php index 9649d37..99dbda4 100644 --- a/HtmlNode.php +++ b/HtmlNode.php @@ -549,3 +554,12 @@ class HtmlNode { - return $this->find($selector, $idx, $lowercase) ?: null; + if(!$element = $this->find($selector, $idx, $lowercase))...

olalav posted a comment on ticket #204

olalav — Wed, 07 Dec 2022 09:04:40 -0000

diff --git a/HtmlNode.php b/HtmlNode.php index 9649d37..aef8b17 100644 --- a/HtmlNode.php +++ b/HtmlNode.php @@ -549,3 +549,12 @@ class HtmlNode + function first($selector, $idx = 0, $lowercase = false) + { + return $this->expect($selector, $idx, $lowercase); + } Missed semicolon and preformatting.

olalav created ticket #204

olalav — Wed, 07 Dec 2022 09:01:04 -0000

Convenience function for getting first element

olalav posted a comment on ticket #201

olalav — Wed, 07 Dec 2022 08:54:29 -0000

diff --git a/simple_html_dom.php b/simple_html_dom.php index bce4d9e..97d6e1d 100644 --- a/simple_html_dom.php +++ b/simple_html_dom.php @@ -117,3 +117,3 @@ function file_get_html( $dom->clear(); - return false; + $contents = ""; } @@ -144,5 +144,4 @@ function str_get_html( $dom->clear(); - return false; + $contents = ""; } - return $dom->load($str, $lowercase, $stripRN); Better version with tabs. PS! The inline editor and preview function on the site seems to hide the first line of content :|

olalav created ticket #203

olalav — Wed, 07 Dec 2022 08:51:07 -0000

Always tell user where he expected non-existing element

Cees van Wageningen created ticket #202

Cees van Wageningen — Thu, 24 Nov 2022 10:05:45 -0000

Preg_match error occurs after saving new contentblock

olalav created ticket #201

olalav — Tue, 11 Oct 2022 07:22:19 -0000

Never return false on documents

Jim Longo posted a comment on ticket #60

Jim Longo — Tue, 30 Aug 2022 21:21:45 -0000

I see. One would use $e->innertext to get the text inside the tag.

Jim Longo created ticket #60

Jim Longo — Mon, 29 Aug 2022 23:12:25 -0000

No anchor text returned

LogMANOriginal modified ticket #200

LogMANOriginal — Mon, 09 May 2022 18:01:28 -0000

"Creation of dynamic property" warning in PHP 8.2 (version 1.9.1)

LogMANOriginal posted a comment on ticket #200

LogMANOriginal — Mon, 09 May 2022 18:01:14 -0000

Thanks for your bug report. This is actually a typo. The variable should be called $optional_closing_tags. There is a recent commit in master that illustrates the fix. This should also work in PHP 8.2 and higher. [8dc21bcb714c4edcb4318bdc3f198f4f78762381]

Jeffrey Kastner modified a comment on ticket #159

Jeffrey Kastner — Thu, 05 May 2022 23:22:57 -0000

disregard

Jeffrey Kastner modified a comment on ticket #159

Jeffrey Kastner — Thu, 05 May 2022 21:47:24 -0000

Jeffrey Kastner modified a comment on ticket #159

Jeffrey Kastner — Thu, 05 May 2022 21:42:44 -0000

If I am understanding attribute selectors, this is not working again.. * ^ and $ all return 2 Example; echo count( str_get_html('Hello!ME OH MI!')->find('span[class^=second]') ); I have been trying to use attribute selectors to try and 'find' a div with an id with random numbers and -slideshow for the value (ex. 8099435804-slideshow) and I haven't been able to get it to work. ~in my case it returns all div's in the...

Jeffrey Kastner posted a comment on ticket #159

Jeffrey Kastner — Thu, 05 May 2022 21:31:20 -0000

Bjørn Rosell created ticket #200

Bjørn Rosell — Mon, 02 May 2022 11:59:30 -0000

"Creation of dynamic property" warning in PHP 8.2 (version 1.9.1)

LogMANOriginal modified ticket #199

LogMANOriginal — Mon, 25 Apr 2022 19:32:41 -0000

Incorrect handling of
tags next to line breaks

LogMANOriginal posted a comment on ticket #199

LogMANOriginal — Mon, 25 Apr 2022 19:32:30 -0000

Looks good now! However, you must set the Unicode flag, or else preg_replace() may return an invalid Unicode string, which may cause the second preg_replace() to return NULL, and a deprecation error for the third preg_replace(). Good catch. Fixed via [b8d048e46b7f1964c28ea041d39ccb1d05f9a0ed]. And about the manual: I see now that the navigation sidebar is aligned far down upon page load, so that only the documentation for the functions (isset etc.) is immediately visible, not the more useful "Quick...

LogMANOriginal committed [b8d048]

LogMANOriginal — Mon, 25 Apr 2022 19:25:17 -0000

HtmlNode: Replace and collapse unicode whitespace in plaintext

olalav modified a comment on ticket #199

olalav — Sun, 24 Apr 2022 19:48:53 -0000

Looks good now! However, you must set the Unicode flag, or else preg_replace() may return an invalid Unicode string, which may cause the second preg_replace() to return NULL, and a deprecation error for the third preg_replace(). diff --git a/HtmlNode.php b/HtmlNode.php index 9bc6a1a..9649d37 100644 --- a/HtmlNode.php +++ b/HtmlNode.php @@ -351 +351 @@ class HtmlNode - $ret = preg_replace('/\s+/', ' ', $ret); + $ret = preg_replace('/\s+/u', ' ', $ret); And about the manual: I see now that the navigation...

olalav posted a comment on ticket #199

olalav — Sun, 24 Apr 2022 19:38:59 -0000

Looks good now! However, you must set the Unicode flag here or else preg_replace() returns NULL for certain strings, which causes (deprecation) errors further down. diff --git a/HtmlNode.php b/HtmlNode.php index 9bc6a1a..9649d37 100644 --- a/HtmlNode.php +++ b/HtmlNode.php @@ -351 +351 @@ class HtmlNode - $ret = preg_replace('/\s+/', ' ', $ret); + $ret = preg_replace('/\s+/u', ' ', $ret); And about the manual: I see now that the navigation sidebar is aligned far down upon page load, so that only...

LogMANOriginal posted a comment on ticket #199

LogMANOriginal — Sun, 24 Apr 2022 16:47:13 -0000

PS! Would be nice if you could link to the manual from the "Support" section, because it was hard to find. https://sourceforge.net/projects/simplehtmldom/support Turns out that page is managed by SF. There is no way to change the contents of that page 😔 I added a "Manual" tab instead.

LogMANOriginal posted a comment on ticket #199

LogMANOriginal — Sun, 24 Apr 2022 16:34:09 -0000

PS! Would be nice if you could link to the manual from the "Support" section, because it was hard to find. https://sourceforge.net/projects/simplehtmldom/support Good idea! I'll do that. The space thing works now, but the BR tag is still not handled well. Try the code in the original post and compare the output when (un)-commenting the commented line. I'm comparing the output of plaintext with what is displayed in the browser and it looks exactly the same. Please note that I have removed wordwrap()...

olalav posted a comment on ticket #199

olalav — Sun, 24 Apr 2022 15:32:22 -0000

The space thing works now, but the BR tag is still not handled well. Try the code in the original post and compare the output when (un)-commenting the commented line. PS! Would be nice if you could link to the manual from the "Support" section, because it was hard to find. https://sourceforge.net/projects/simplehtmldom/support

LogMANOriginal posted a comment on ticket #199

LogMANOriginal — Sun, 24 Apr 2022 15:03:17 -0000

[67c0f4e21091a9cc66151610a653724a0acb1b69] fixes the whitespace issue. Let me know if this works for you.

LogMANOriginal committed [67c0f4]

LogMANOriginal — Sun, 24 Apr 2022 15:02:17 -0000

HtmlNode: Replace and collapse whitespace in plaintext

LogMANOriginal posted a comment on ticket #199

LogMANOriginal — Sun, 24 Apr 2022 14:25:54 -0000

Shouldn't plaintext convert newlines to spaces? Did you change this recently? Surely this is a bug/regression, or am I missing something completely? The plaintext implementation is completely rewritten but it passes all tests. Your particular case probably isn't covered by any of the tests right now. I'll check this as well. At the very least
seems to work right. PS! Where on the SourceForge page is the link to the manual (the one with the clickable tabs with examples, etc.)? I hope you didn't...

olalav posted a comment on ticket #199

olalav — Sun, 24 Apr 2022 13:19:37 -0000

Shouldn't plaintext convert newlines to spaces? Did you change this recently? Surely this is a bug/regression, or am I missing something completely? $text = "

Hello" . "\n" . "World

"; $plain = str_get_html($text)->plaintext; echo "PLAINTEXT:\n" . $plain . "\n\n"; echo "WORDWRAP:\n" . wordwrap($plain, 80) . "\n"; PS! Where on the SourceForge page is the link to the manual (the one with the clickable tabs with examples, etc.)? I hope you didn't remove this, because I use it as a reference all...

LogMANOriginal posted a comment on ticket #199

LogMANOriginal — Sun, 24 Apr 2022 08:56:31 -0000

Please try again with current master. From what I can tell, the output looks right: ***** ** ********,*** ****** ********. ******* **** *** ** ** ******* *** *** ****** ***. *** **** *** ****** ****. *.***. ** **** *** ***** **** ***** ********* *** ************ ** *** *** *** *** ** ****. *** ** ****, ******** ******* *** ******** ********** * ** ******* ****. ******** ** *** *** **** ***********, ** *** *** *** * *** *** ********* .*** ***** ******* *** ** **** ***.*** ****** *** ** **** *** ********...

LogMANOriginal modified ticket #198

LogMANOriginal — Sun, 24 Apr 2022 08:45:22 -0000

iconv() detected an illegal character in input string

LogMANOriginal posted a comment on ticket #198

LogMANOriginal — Sun, 24 Apr 2022 08:45:22 -0000

This is fixed via [c53a612e6fe61d5b1efc0c3270e20aa34e4e84ee]. Instead of using //IGNORE, it needs to be wrapped inside a try-catch block, so that the character set is detected properly. Eventually, this will be replaced by a better solution, but this works for now. Thanks again for reporting!

LogMANOriginal committed [c53a61]