Full-Text RSS included as a script instead of file_get_contents call. Tnx to @Faless. Fix issues #366 and #463 #472
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
a few words why these errors happen (i.e. why it is better to call script inline), on my opinion. My view is, that problem is often not in php settings, but a level up: in virtual host config (like with digest auth) or even with firewall settings: result is impossibility to call makefulltextfeed.php via http from running script. This makes the problem hard to repeat and investigate. With this fix we eliminate this problem.
Small note: this code is (a bit like) workaround, and if you plan long life for v.1, maybe, there is a reason to make it more native (important word here is "maybe", we can always refactor it later).