friendica/friendica-addons

Nsfw showmore #69

Merged

CyberDomovoy merged 4 commits from nsfw_showmore into master

2012-07-24 02:39:20 +02:00

CyberDomovoy commented

2012-07-23 20:26:13 +02:00

(Migrated from github.com)

Trouble: it does need the modifications from CyberDomovoy/friendica@35f1081bfe

I'm not used to git yet, i need to learn how to commit and do pull requests nicely; sorry.

Trouble: it does need the modifications from CyberDomovoy/friendica@35f1081bfe834d268744e091c24d730cda2064c4 I'm not used to git yet, i need to learn how to commit and do pull requests nicely; sorry.

CyberDomovoy commented

2012-07-23 20:57:34 +02:00

(Migrated from github.com)

Hmmm, tested it on an older php version, with xdebug installed.
DomDocument produces a lot of warnings.

This is not ready to pull.

Hmmm, tested it on an older php version, with xdebug installed. DomDocument produces a lot of warnings. This is not ready to pull.

CyberDomovoy commented

2012-07-23 21:33:26 +02:00

(Migrated from github.com)

I prevented DomDocument::loadHTML to emit warnings, so it now appears to work ok.

Anyway, i don't really like this solution, those warnings are there for a reason, it would be better to make sure we have proper html code produced by the templates.

I prevented DomDocument::loadHTML to emit warnings, so it now appears to work ok. Anyway, i don't really like this solution, those warnings are there for a reason, it would be better to make sure we have proper html code produced by the templates.

Sign in to join this conversation.

No reviewers

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: friendica/friendica-addons#69

No description provided.