Configuration option for _cleanClasses about readability.php HOT 8 CLOSED

What do you mean? Which classes do you want to clean and why?

from readability.php.

I mean this method

// by default can be set to true
if ($this->configuration->getCleanClasses())  {
    $this->_cleanClasses($article);
}

from readability.php.

Alright, now I get you. What would be the purpose of making this optional? I think the library's ultimate functionality is to return the cleanest text possible. Why'd you want to keep the original classes which would point to inexistent styles?

from readability.php.

Sometimes the result may be unsatisfactory, and I rely on these classes when post-processing the result.
I can create some rules for thу concrete url-sources, relying on css classes and ids of the original html.

from readability.php.

I clean out these classes myself, in any case, but a little later, after processing additional rules.

from readability.php.

Mmmm, I understand. Can you provide an example for an "unsatisfactoy result"? As I always answered to this type of requests, I prefer to create a more intelligent algorithm rather than giving the user a framework to tweak the functionality.

from readability.php.

Yeah. I'll do it some later.
Maybe it will be betted to submit a failing test case )

But this config option will not interfere in any way - it will give more flexibility )

from readability.php.

I get what you mean, but this kind of changes allows everyone to request something to tweak the intended functionality. Thanks for understanding. Will wait for your failing test case :)

from readability.php.