Merge pull request #159 from ctoth/master

More small English cleanups

flask_admin/actions.py

@@ -29,10 +29,10 @@ class ActionsMixin(object):
         Actions mixin.
 
         In some cases, you might work with more than one "entity" (model, file, etc) in
-        your admin view and will want to perform actions on group of entities at once.
+        your admin view and will want to perform actions on a group of entities simultaneously.
 
         In this case, you can add this functionality by doing this:
-        1. Add mixin to your administrative view class
+        1. Add this mixin to your administrative view class
         2. Call `init_actions` in your class constructor
         3. Expose actions view
         4. Import `actions.html` library and add call library macros in your template
@@ -76,7 +76,7 @@ class ActionsMixin(object):
 
     def get_actions_list(self):
         """
-            Return list and a dictionary of allowed actions.
+            Return a list and a dictionary of allowed actions.
         """
         actions = []
         actions_confirmation = {}

flask_admin/contrib/fileadmin.py

@@ -162,7 +162,7 @@ class FileAdmin(BaseView, ActionsMixin):
             :param base_url:
                 Base URL for the files
             :param name:
-                Name of this view. If not provided, will be defaulted to the class name.
+                Name of this view. If not provided, will default to the class name.
             :param category:
                 View category
             :param endpoint:
@@ -171,7 +171,7 @@ class FileAdmin(BaseView, ActionsMixin):
                 URL for view
             :param verify_path:
                 Verify if path exists. If set to `True` and path does not exist
-                will throw exception.
+                will raise an exception.
         """
         self.base_path = base_path
         self.base_url = base_url
@@ -198,7 +198,7 @@ class FileAdmin(BaseView, ActionsMixin):
 
     def is_accessible_path(self, path):
         """
-            Verify if path is accessible for current user.
+            Verify if the provided path is accessible for the current user.
 
             Override to customize behavior.
 
@@ -242,7 +242,7 @@ class FileAdmin(BaseView, ActionsMixin):
 
     def is_file_editable(self, filename):
         """
-            Verify if file can be edited.
+            Determine if the file can be edited.
 
             Override to customize behavior.
 
@@ -261,7 +261,7 @@ class FileAdmin(BaseView, ActionsMixin):
 
     def is_in_folder(self, base_path, directory):
         """
-            Verify if `directory` is in `base_path` folder
+            Verify that `directory` is in `base_path` folder
 
             :param base_path:
                 Base directory path
@@ -319,9 +319,9 @@ class FileAdmin(BaseView, ActionsMixin):
         """
             Verify and normalize path.
 
-            If path is not relative to the base directory, will throw 404 exception.
+            If the path is not relative to the base directory, will raise a 404 exception.
 
-            If path does not exist, will also throw 404 exception.
+            If the path does not exist, this will also raise a 404 exception.
         """
         base_path = self.get_base_path()
 
@@ -353,7 +353,7 @@ class FileAdmin(BaseView, ActionsMixin):
             Index view method
 
             :param path:
-                Optional directory path. If not provided, will use base directory
+                Optional directory path. If not provided, will use the base directory
         """
         # Get path and verify if it is valid
         base_path, directory, path = self._normalize_path(path)
@@ -406,7 +406,7 @@ class FileAdmin(BaseView, ActionsMixin):
             Upload view method
 
             :param path:
-                Optional directory path. If not provided, will use base directory
+                Optional directory path. If not provided, will use the base directory
         """
         # Get path and verify if it is valid
         base_path, directory, path = self._normalize_path(path)
@@ -439,7 +439,7 @@ class FileAdmin(BaseView, ActionsMixin):
             Directory creation view method
 
             :param path:
-                Optional directory path. If not provided, will use base directory
+                Optional directory path. If not provided, will use the base directory
         """
         # Get path and verify if it is valid
         base_path, directory, path = self._normalize_path(path)

flask_admin/contrib/mongoengine/view.py

@@ -75,7 +75,7 @@ class ModelView(BaseModelView):
     """
         Field to filter converter.
 
-        Override this attribute to use non-default converter.
+        Override this attribute to use a non-default converter.
     """
 
     column_type_formatters = DEFAULT_FORMATTERS
@@ -123,7 +123,7 @@ class ModelView(BaseModelView):
 
     def get_pk_value(self, model):
         """
-            Return primary key value from the model instance
+            Return the primary key value from the model instance
 
             :param model:
                 Model instance
@@ -154,7 +154,7 @@ class ModelView(BaseModelView):
 
     def scaffold_sortable_columns(self):
         """
-            Return sortable columns dictionary (name, field)
+            Return a dictionary of sortable columns (name, field)
         """
         columns = {}
 
@@ -222,7 +222,7 @@ class ModelView(BaseModelView):
 
     def is_valid_filter(self, filter):
         """
-            Validate if it is valid MongoEngine filter
+            Validate if the provided filter is a valid MongoEngine filter
 
             :param filter:
                 Filter object
@@ -254,7 +254,7 @@ class ModelView(BaseModelView):
             :param search:
                 Search criteria
             :param filters:
-                List of applied fiters
+                List of applied filters
             :param execute:
                 Run query immediately or not
         """
@@ -307,7 +307,7 @@ class ModelView(BaseModelView):
 
     def get_one(self, id):
         """
-            Return single model instance by ID
+            Return a single model instance by its ID
 
             :param id:
                 Model ID

flask_admin/contrib/sqlamodel/view.py

@@ -18,7 +18,7 @@ from .typefmt import DEFAULT_FORMATTERS
 
 class ModelView(BaseModelView):
     """
-        SQLALchemy model view
+        SQLAlchemy model view
 
         Usage sample::
 
@@ -90,7 +90,7 @@ class ModelView(BaseModelView):
             class MyModelView(ModelView):
                 column_searchable_list = (User.name, User.email)
 
-        Following search rules apply:
+        The following search rules apply:
 
         - If you enter *ZZZ* in the UI search field, it will generate *ILIKE '%ZZZ%'*
           statement against searchable columns.
@@ -103,8 +103,8 @@ class ModelView(BaseModelView):
         - If you prefix your search term with ^, it will find all rows
           that start with ^. So, if you entered *^ZZZ*, *ILIKE 'ZZZ%'* will be used.
 
-        - If you prefix your search term with =, it will do exact match.
-          For example, if you entered *=ZZZ*, *ILIKE 'ZZZ'* statement will be used.
+        - If you prefix your search term with =, it will perform an exact match.
+          For example, if you entered *=ZZZ*, the statement *ILIKE 'ZZZ'* will be used.
     """
 
     column_filters = None
@@ -163,11 +163,11 @@ class ModelView(BaseModelView):
     """
         If set to `False` and user deletes more than one model using built in action,
         all models will be read from the database and then deleted one by one
-        giving SQLAlchemy chance to manually cleanup any dependencies (many-to-many
+        giving SQLAlchemy a chance to manually cleanup any dependencies (many-to-many
         relationships, etc).
 
-        If set to `True`, will run `DELETE` statement which is somewhat faster,
-        but might leave corrupted data if you forget to configure `DELETE
+        If set to `True`, will run a `DELETE` statement which is somewhat faster,
+        but may leave corrupted data if you forget to configure `DELETE
         CASCADE` for your model.
     """
 
@@ -195,9 +195,9 @@ class ModelView(BaseModelView):
             class MyModelView(ModelView):
                 inline_models = (MyInlineModelForm(MyInlineModel),)
 
-        You can customize generated field name by:
+        You can customize the generated field name by:
 
-        1. Using `form_name` property as option:
+        1. Using the `form_name` property as a key to the options dictionary:
 
             class MyModelView(ModelView):
                 inline_models = ((Post, dict(form_label='Hello')))
@@ -226,15 +226,15 @@ class ModelView(BaseModelView):
             :param model:
                 Model class
             :param session:
-                SQLALchemy session
+                SQLAlchemy session
             :param name:
-                View name. If not set, will default to model name
+                View name. If not set, defaults to the model name
             :param category:
                 Category name
             :param endpoint:
-                Endpoint name. If not set, will default to model name
+                Endpoint name. If not set, defaults to the model name
             :param url:
-                Base URL. If not set, will default to '/admin/' + endpoint
+                Base URL. If not set, defaults to '/admin/' + endpoint
         """
         self.session = session
 
@@ -270,19 +270,19 @@ class ModelView(BaseModelView):
     # Scaffolding
     def scaffold_pk(self):
         """
-            Return primary key name from a model
+            Return the primary key name from a model
         """
         return tools.get_primary_key(self.model)
 
     def get_pk_value(self, model):
         """
-            Return PK value from a model object.
+            Return the PK value from a model object.
         """
         return getattr(model, self._primary_key)
 
     def scaffold_list_columns(self):
         """
-            Return list of columns from the model.
+            Return a list of columns from the model.
         """
         columns = []
 
@@ -307,7 +307,7 @@ class ModelView(BaseModelView):
 
     def scaffold_sortable_columns(self):
         """
-            Return dictionary of sortable columns.
+            Return a dictionary of sortable columns.
             Key is column name, value is sort column/field.
         """
         columns = dict()
@@ -322,7 +322,7 @@ class ModelView(BaseModelView):
 
                 column = p.columns[0]
 
-                # Can't sort by on primary and foreign keys by default
+                # Can't sort on primary or foreign keys by default
                 if column.foreign_keys:
                     continue
 
@@ -383,7 +383,7 @@ class ModelView(BaseModelView):
 
     def is_text_column_type(self, name):
         """
-            Verify if column type is text-based.
+            Verify if the provided column type is text-based.
 
             :returns:
                 ``True`` for ``String``, ``Unicode``, ``Text``, ``UnicodeText``
@@ -479,7 +479,7 @@ class ModelView(BaseModelView):
 
     def is_valid_filter(self, filter):
         """
-            Verify that provided filter object is derived from the
+            Verify that the provided filter object is derived from the
             SQLAlchemy-compatible filter class.
 
             :param filter:
@@ -522,7 +522,7 @@ class ModelView(BaseModelView):
 
     def scaffold_auto_joins(self):
         """
-            Return list of joined tables by going through the
+            Return a list of joined tables by going through the
             displayed columns.
         """
         if not self.column_auto_select_related:
@@ -669,7 +669,7 @@ class ModelView(BaseModelView):
 
     def get_one(self, id):
         """
-            Return one model by its id.
+            Return a single model by its id.
 
             :param id:
                 Model id

flask_admin/tools.py

@@ -30,7 +30,7 @@ def import_attribute(name):
         :param name:
             String reference.
 
-        Throws ImportError or AttributeError if module or attribute do not exist.
+        Raises ImportError or AttributeError if module or attribute do not exist.
 
         Example::
 
@@ -80,7 +80,7 @@ def rec_getattr(obj, attr, default=None):
 
 def get_dict_attr(obj, attr, default=None):
     """
-        Get attibute of the object without triggering its __getattr__.
+        Get attribute of the object without triggering its __getattr__.
 
         :param obj:
             Object
@@ -98,7 +98,7 @@ def get_dict_attr(obj, attr, default=None):
 
 def get_property(obj, name, old_name, default=None):
     """
-        Check if old property name exists and if it is - show warning message
+        Check if old property name exists and if it does - show warning message
         and return value.
 
         Otherwise, return new property value