Issue #2636342 by AndyF, MegaChriz: Improved documentation of FeedsConfigurable::__get().

includes/FeedsConfigurable.inc

@@ -15,6 +15,10 @@ class FeedsNotExistingException extends Exception {
  * Base class for configurable classes. Captures configuration handling, form
  * handling and distinguishes between in-memory configuration and persistent
  * configuration.
+ *
+ * Due to the magic method __get(), protected properties from this class and
+ * from classes that extend this class will be publicly readable (but not
+ * writeable).
  */
 abstract class FeedsConfigurable {
 
@@ -139,8 +143,13 @@ abstract class FeedsConfigurable {
   }
 
   /**
-   * Override magic method __get(). Make sure that $this->config goes through
-   * getConfig().
+   * Overrides magic method __get().
+   *
+   * - Makes sure that external reads of FeedsConfigurable::config go through
+   *   ::getConfig();
+   * - Makes private and protected properties from this class and protected
+   *   properties from child classes publicly readable.
+   * - Prevents warnings when accessing non-existent properties.
    */
   public function __get($name) {
     if ($name == 'config') {