The Fast News feature doesn't exist. Here is a patch that removes the respective part from the README.txt.

CommentFileSizeAuthor
#5 interdiff-2690169-2-4.txt1.98 KBmegachriz
#5 feeds-update-readme-2690169-4.patch2.73 KBmegachriz
#2 feeds_update-readme_2690169.patch3.06 KBtterranigma
feeds_fast-news-removed_1401846_0.patch827 bytestterranigma

Comments

tterranigma created an issue. See original summary.

tterranigma’s picture

tterranigma commented
Title: Remove Feeds Fast News from the README.txt » Update README.txt
Category: Bug report » Task
StatusFileSize
new feeds_update-readme_2690169.patch3.06 KB

Actually, the included with the module README.txt is the old README.txt from the Drupal 6 version as far as I can tell. I tried to update it, however it surely needs a look from the devs, especially on the hidden variables (two of them are missing their description).

megachriz’s picture

megachriz
Primary language Dutch
commented

Good idea to update the README.

  1. +++ b/README.txt
@@ -178,6 +170,10 @@ Default:     FALSE
+Name:        feeds_library_dir
+Default:     FALSE
+Description:
+

@@ -186,18 +182,15 @@ Name:        feeds_source_class
+Name:        feeds_reschedule
+Default:     FALSE
+Description:
+

    We should indeed look up what these two variables exactly do. 

  2. +++ b/README.txt
@@ -215,6 +208,14 @@ Description: The extension mbstring is used to convert encodings during parsing.
+Name:        feeds_tests_feeds_after_parse_empty_items
+Default:     FALSE
+Description: If set to TRUE, the list of items to import is emptied in case the test says that there are
+             items in there with encoding issues. These items can not be processed during
+             tests without having a test failure because in < PHP 5.4 that would produce
+             the following warning:
+               htmlspecialchars(): Invalid multibyte sequence in argument
+

    This is only used during tests and only used by the feeds_tests module. Therefore, I don't think that this variable should be documented in the README.

tterranigma’s picture

tterranigma commented

Actually, the 'Testing' section in the README includes only a link to the documentation. So, to keep it consistent, we can omit the feeds_tests_feeds_after_parse_empty_items variable in the README and update the documentation at http://drupal.org/node/622700 (which could use some refactoring).

megachriz’s picture

megachriz
Primary language Dutch
commented
StatusFileSize
new feeds-update-readme-2690169-4.patch2.73 KB
new interdiff-2690169-2-4.txt1.98 KB

Wow, talk about outdated. Never knew that there was an installation testing profile for Feeds. I don't think such a profile is needed anymore. I have run tests fine all the time since I started with developing on Feeds (now about three years ago). Let's remove that section from the README.

I'm not sure if the "feeds_reschedule" should be documented in the README. I found out it is more or less a status variable: it determines which importers should be rescheduled. I think it can lead to unexpected results if that variable is permanently overridden in settings.php.

  • MegaChriz committed 3cd0b27 on 7.x-2.x 
    Issue #2690169 by tterranigma, MegaChriz: updated README.txt: documented...
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Needs review » Fixed

Committed #5.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.