Skip to content

Refactoring#916

Merged
jtojnar merged 13 commits into
masterfrom
refactor
Feb 8, 2019
Merged

Refactoring#916
jtojnar merged 13 commits into
masterfrom
refactor

Conversation

@jtojnar

@jtojnar jtojnar commented Apr 14, 2017

Copy link
Copy Markdown
Member

I have been trying to use null instead of false or '' whenever possible. Additionally, I also changed majority of comparisons to strict version, filled in missing parameters to phpdoc and also added array type hints.

Tests should be merged first.

Comment thread spouts/youtube/youtube.php Outdated
@jtojnar jtojnar force-pushed the refactor branch 5 times, most recently from 6dccd93 to 63d2408 Compare April 15, 2017 19:48
@jtojnar jtojnar changed the title Refactoring [WIP] Refactoring Apr 21, 2017
@jtojnar jtojnar force-pushed the refactor branch 2 times, most recently from f5da4a5 to d6c0d9d Compare April 24, 2017 23:39
@jtojnar jtojnar force-pushed the refactor branch 2 times, most recently from f145a6b to 0ec02ab Compare May 7, 2017 11:13
@jtojnar

jtojnar commented May 8, 2017

Copy link
Copy Markdown
Member Author

For some reason https://feeds.feedburner.com/sandraandwoo now uses feedburner favicon instead of the site one. Hopefully, this will be addressed by a rewrite of fetchFavicon method.

@niol

niol commented May 8, 2017

Copy link
Copy Markdown
Collaborator

Yes this happens since merging #920 .

@jtojnar

jtojnar commented May 8, 2017

Copy link
Copy Markdown
Member Author

You are right, I was probably using old master on my selfoss instance. For now, I have reverted one of the commits, see #924.

@jtojnar jtojnar force-pushed the refactor branch 2 times, most recently from fb325ea to 1afe4d6 Compare May 10, 2017 15:50
@jtojnar jtojnar force-pushed the refactor branch 4 times, most recently from 3e07bae to 986f814 Compare June 15, 2017 23:51
@jtojnar jtojnar force-pushed the refactor branch 3 times, most recently from a6bd0d1 to cc64077 Compare June 28, 2017 00:18
@jtojnar jtojnar force-pushed the refactor branch 2 times, most recently from 883e795 to 47f251b Compare March 9, 2018 04:21
@jtojnar jtojnar merged commit 84332df into master Feb 8, 2019
@jtojnar jtojnar changed the title [WIP] Refactoring Refactoring Feb 8, 2019
@jtojnar jtojnar deleted the refactor branch February 8, 2019 18:00
@jtojnar jtojnar added this to the 2.19 milestone Apr 3, 2019
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