xbanners\models\Map\BannerImageI18nTableMap

Complexity

Total Complexity

81

Size/Duplication

Total Lines	484
Duplicated Lines	8.88 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	buildRelations()	10	10	1
B	addInstanceToPool()	9	9	9
A	getOMClass()	0	4	2
A	addSelectColumns()	0	18	2
A	getTableMap()	0	4	1
A	buildTableMap()	7	7	2
A	doDeleteAll()	0	4	1
A	initialize()	17	17	1
C	removeInstanceFromPool()	0	21	21
B	getPrimaryKeyHashFromRow()	0	9	21
A	getPrimaryKeyFromRow()	0	17	3
A	populateObject()	0	18	2
B	populateObjects()	0	25	3
D	doDelete()	0	39	9
A	doInsert()	0	22	3

<?php

namespace xbanners\models\Map;

use Propel\Runtime\Propel;
use Propel\Runtime\ActiveQuery\Criteria;
use Propel\Runtime\ActiveQuery\InstancePoolTrait;
use Propel\Runtime\Connection\ConnectionInterface;
use Propel\Runtime\DataFetcher\DataFetcherInterface;
use Propel\Runtime\Exception\PropelException;
use Propel\Runtime\Map\RelationMap;
use Propel\Runtime\Map\TableMap;
use Propel\Runtime\Map\TableMapTrait;
use xbanners\models\BannerImageI18n;
use xbanners\models\BannerImageI18nQuery;


/**
 * This class defines the structure of the 'banner_image_i18n' table.
 *
 *
 *
 * This map class is used by Propel to do runtime db structure discovery.

There must be exactly one blank line between descriptions in a doc comment

 * For example, the createSelectSql() method checks the type of a given column used in an
 * ORDER BY clause to know whether it needs to apply SQL to make the ORDER BY case-insensitive
 * (i.e. if it's a text column type).
 *
 */

Additional blank lines found at end of doc comment

class BannerImageI18nTableMap extends TableMap
{

Opening brace of a class must be on the same line as the definition

    use InstancePoolTrait;
    use TableMapTrait;

    /**
     * The (dot-path) name of this class
     */
    const CLASS_NAME = 'xbanners.models.Map.BannerImageI18nTableMap';

    /**
     * The default database name for this class
     */
    const DATABASE_NAME = 'Shop';

    /**
     * The table name for this class
     */
    const TABLE_NAME = 'banner_image_i18n';

    /**
     * The related Propel class for this table
     */
    const OM_CLASS = '\\xbanners\\models\\BannerImageI18n';

    /**
     * A class that can be returned by this tableMap
     */
    const CLASS_DEFAULT = 'xbanners.models.BannerImageI18n';

    /**
     * The total number of columns
     */
    const NUM_COLUMNS = 6;

    /**
     * The number of lazy-loaded columns
     */
    const NUM_LAZY_LOAD_COLUMNS = 0;

    /**
     * The number of columns to hydrate (NUM_COLUMNS - NUM_LAZY_LOAD_COLUMNS)
     */
    const NUM_HYDRATE_COLUMNS = 6;

    /**
     * the column name for the id field
     */
    const COL_ID = 'banner_image_i18n.id';

    /**
     * the column name for the locale field
     */
    const COL_LOCALE = 'banner_image_i18n.locale';

    /**
     * the column name for the src field
     */
    const COL_SRC = 'banner_image_i18n.src';

    /**
     * the column name for the name field
     */
    const COL_NAME = 'banner_image_i18n.name';

    /**
     * the column name for the clicks field
     */
    const COL_CLICKS = 'banner_image_i18n.clicks';

    /**
     * the column name for the description field
     */
    const COL_DESCRIPTION = 'banner_image_i18n.description';

    /**
     * The default string format for model objects of the related table
     */
    const DEFAULT_STRING_FORMAT = 'YAML';

    /**
     * holds an array of fieldnames
     *
     * first dimension keys are the type constants

Doc comment long description must start with a capital letter

     * e.g. self::$fieldNames[self::TYPE_PHPNAME][0] = 'Id'
     */
    protected static $fieldNames = array (

There must be no space between the "array" keyword and the opening parenthesis

        self::TYPE_PHPNAME       => array('Id', 'Locale', 'Src', 'Name', 'Clicks', 'Description', ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_CAMELNAME     => array('id', 'locale', 'src', 'name', 'clicks', 'description', ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_COLNAME       => array(BannerImageI18nTableMap::COL_ID, BannerImageI18nTableMap::COL_LOCALE, BannerImageI18nTableMap::COL_SRC, BannerImageI18nTableMap::COL_NAME, BannerImageI18nTableMap::COL_CLICKS, BannerImageI18nTableMap::COL_DESCRIPTION, ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

As per coding style, self should be used for accessing local static members.

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_FIELDNAME     => array('id', 'locale', 'src', 'name', 'clicks', 'description', ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_NUM           => array(0, 1, 2, 3, 4, 5, )

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

    );

    /**
     * holds an array of keys for quick access to the fieldnames array
     *
     * first dimension keys are the type constants

Doc comment long description must start with a capital letter

     * e.g. self::$fieldKeys[self::TYPE_PHPNAME]['Id'] = 0
     */
    protected static $fieldKeys = array (

There must be no space between the "array" keyword and the opening parenthesis

        self::TYPE_PHPNAME       => array('Id' => 0, 'Locale' => 1, 'Src' => 2, 'Name' => 3, 'Clicks' => 4, 'Description' => 5, ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_CAMELNAME     => array('id' => 0, 'locale' => 1, 'src' => 2, 'name' => 3, 'clicks' => 4, 'description' => 5, ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_COLNAME       => array(BannerImageI18nTableMap::COL_ID => 0, BannerImageI18nTableMap::COL_LOCALE => 1, BannerImageI18nTableMap::COL_SRC => 2, BannerImageI18nTableMap::COL_NAME => 3, BannerImageI18nTableMap::COL_CLICKS => 4, BannerImageI18nTableMap::COL_DESCRIPTION => 5, ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

As per coding style, self should be used for accessing local static members.

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_FIELDNAME     => array('id' => 0, 'locale' => 1, 'src' => 2, 'name' => 3, 'clicks' => 4, 'description' => 5, ),

If the line declaring an array spans longer than 80 characters, each element should be broken into its own line

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

        self::TYPE_NUM           => array(0, 1, 2, 3, 4, 5, )

Comma not allowed after last value in single-line array declaration

There should be no white space before a closing ")"

    );

    /**
     * Initialize the table attributes and columns
     * Relations are not initialized by this method since they are lazy loaded
     *
     * @return void

If there is no return value for a function, there must not be a @return tag.

     * @throws PropelException

Comment missing or not on the next line for @throws tag in function comment

     */
    public function initialize()
    {
        // attributes
        $this->setName('banner_image_i18n');
        $this->setPhpName('BannerImageI18n');
        $this->setIdentifierQuoting(false);
        $this->setClassName('\\xbanners\\models\\BannerImageI18n');
        $this->setPackage('xbanners.models');
        $this->setUseIdGenerator(false);
        // columns
        $this->addForeignPrimaryKey('id', 'Id', 'INTEGER' , 'banner_image', 'id', true, 11, null);

Space found before comma in function call

        $this->addPrimaryKey('locale', 'Locale', 'VARCHAR', true, 5, 'ru');
        $this->addColumn('src', 'Src', 'VARCHAR', false, 255, null);
        $this->addColumn('name', 'Name', 'VARCHAR', false, 255, null);
        $this->addColumn('clicks', 'Clicks', 'INTEGER', false, 20, null);
        $this->addColumn('description', 'Description', 'LONGVARCHAR', false, null, null);
    } // initialize()

    /**
     * Build the RelationMap objects for this table relationships
     */
    public function buildRelations()
    {
        $this->addRelation('BannerImage', '\\xbanners\\models\\BannerImage', RelationMap::MANY_TO_ONE, array (

For multi-line function calls, each argument should be on a separate line.

There must be no space between the "array" keyword and the opening parenthesis

  0 =>
  array (

There must be no space between the "array" keyword and the opening parenthesis

    0 => ':id',
    1 => ':id',
  ),
), 'CASCADE', null, null, false);

Line indented incorrectly; expected at least 8 spaces, found 0

For multi-line function calls, each argument should be on a separate line.

    } // buildRelations()

    /**
     * Adds an object to the instance pool.
     *
     * Propel keeps cached copies of objects in an instance pool when they are retrieved
     * from the database. In some cases you may need to explicitly add objects
     * to the cache in order to ensure that the same objects are always returned by find*()
     * and findPk*() calls.
     *
     * @param \xbanners\models\BannerImageI18n $obj A \xbanners\models\BannerImageI18n object.
     * @param string $key             (optional) key to use for instance map (for performance boost if key was already calculated externally).
     */
    public static function addInstanceToPool($obj, $key = null)
    {
        if (Propel::isInstancePoolingEnabled()) {
            if (null === $key) {
                $key = serialize([(null === $obj->getId() || is_scalar($obj->getId()) || is_callable([$obj->getId(), '__toString']) ? (string) $obj->getId() : $obj->getId()), (null === $obj->getLocale() || is_scalar($obj->getLocale()) || is_callable([$obj->getLocale(), '__toString']) ? (string) $obj->getLocale() : $obj->getLocale())]);
            } // if key === null
            self::$instances[$key] = $obj;
        }
    }

    /**
     * Removes an object from the instance pool.
     *
     * Propel keeps cached copies of objects in an instance pool when they are retrieved
     * from the database.  In some cases -- especially when you override doDelete
     * methods in your stub classes -- you may need to explicitly remove objects
     * from the cache in order to prevent returning objects that no longer exist.
     *
     * @param mixed $value A \xbanners\models\BannerImageI18n object or a primary key value.
     */
    public static function removeInstanceFromPool($value)
    {
        if (Propel::isInstancePoolingEnabled() && null !== $value) {
            if (is_object($value) && $value instanceof \xbanners\models\BannerImageI18n) {
                $key = serialize([(null === $value->getId() || is_scalar($value->getId()) || is_callable([$value->getId(), '__toString']) ? (string) $value->getId() : $value->getId()), (null === $value->getLocale() || is_scalar($value->getLocale()) || is_callable([$value->getLocale(), '__toString']) ? (string) $value->getLocale() : $value->getLocale())]);

            } elseif (is_array($value) && count($value) === 2) {
                // assume we've been passed a primary key";
                $key = serialize([(null === $value[0] || is_scalar($value[0]) || is_callable([$value[0], '__toString']) ? (string) $value[0] : $value[0]), (null === $value[1] || is_scalar($value[1]) || is_callable([$value[1], '__toString']) ? (string) $value[1] : $value[1])]);
            } elseif ($value instanceof Criteria) {

The class Propel\Runtime\ActiveQuery\Criteria does not exist. Did you forget a USE statement, or did you not list all dependencies?

                self::$instances = [];

                return;
            } else {
                $e = new PropelException("Invalid value passed to removeInstanceFromPool().  Expected primary key or \xbanners\models\BannerImageI18n object; got " . (is_object($value) ? get_class($value) . ' object.' : var_export($value, true)));
                throw $e;
            }

            unset(self::$instances[$key]);
        }
    }

    /**
     * Retrieves a string version of the primary key from the DB resultset row that can be used to uniquely identify a row in this table.
     *
     * For tables with a single-column primary key, that simple pkey value will be returned.  For tables with
     * a multi-column primary key, a serialize()d version of the primary key will be returned.
     *
     * @param array  $row       resultset row.

Expected 1 spaces after parameter type; 2 found

     * @param int    $offset    The 0-based offset for reading from the resultset row.

Expected 1 spaces after parameter type; 4 found

     * @param string $indexType One of the class type constants TableMap::TYPE_PHPNAME, TableMap::TYPE_CAMELNAME
     *                           TableMap::TYPE_COLNAME, TableMap::TYPE_FIELDNAME, TableMap::TYPE_NUM

Parameter comment must end with a full stop

     *
     * @return string The primary key hash of the row

Should the return type not be null|string?

     */
    public static function getPrimaryKeyHashFromRow($row, $offset = 0, $indexType = TableMap::TYPE_NUM)
    {
        // If the PK cannot be derived from the row, return NULL.
        if ($row[TableMap::TYPE_NUM == $indexType ? 0 + $offset : static::translateFieldName('Id', TableMap::TYPE_PHPNAME, $indexType)] === null && $row[TableMap::TYPE_NUM == $indexType ? 1 + $offset : static::translateFieldName('Locale', TableMap::TYPE_PHPNAME, $indexType)] === null) {
            return null;
        }

        return serialize([(null === $row[TableMap::TYPE_NUM == $indexType ? 0 + $offset : static::translateFieldName('Id', TableMap::TYPE_PHPNAME, $indexType)] || is_scalar($row[TableMap::TYPE_NUM == $indexType ? 0 + $offset : static::translateFieldName('Id', TableMap::TYPE_PHPNAME, $indexType)]) || is_callable([$row[TableMap::TYPE_NUM == $indexType ? 0 + $offset : static::translateFieldName('Id', TableMap::TYPE_PHPNAME, $indexType)], '__toString']) ? (string) $row[TableMap::TYPE_NUM == $indexType ? 0 + $offset : static::translateFieldName('Id', TableMap::TYPE_PHPNAME, $indexType)] : $row[TableMap::TYPE_NUM == $indexType ? 0 + $offset : static::translateFieldName('Id', TableMap::TYPE_PHPNAME, $indexType)]), (null === $row[TableMap::TYPE_NUM == $indexType ? 1 + $offset : static::translateFieldName('Locale', TableMap::TYPE_PHPNAME, $indexType)] || is_scalar($row[TableMap::TYPE_NUM == $indexType ? 1 + $offset : static::translateFieldName('Locale', TableMap::TYPE_PHPNAME, $indexType)]) || is_callable([$row[TableMap::TYPE_NUM == $indexType ? 1 + $offset : static::translateFieldName('Locale', TableMap::TYPE_PHPNAME, $indexType)], '__toString']) ? (string) $row[TableMap::TYPE_NUM == $indexType ? 1 + $offset : static::translateFieldName('Locale', TableMap::TYPE_PHPNAME, $indexType)] : $row[TableMap::TYPE_NUM == $indexType ? 1 + $offset : static::translateFieldName('Locale', TableMap::TYPE_PHPNAME, $indexType)])]);
    }

    /**
     * Retrieves the primary key from the DB resultset row
     * For tables with a single-column primary key, that simple pkey value will be returned.  For tables with
     * a multi-column primary key, an array of the primary key columns will be returned.
     *
     * @param array  $row       resultset row.

Expected 1 spaces after parameter type; 2 found

     * @param int    $offset    The 0-based offset for reading from the resultset row.

Expected 1 spaces after parameter type; 4 found

     * @param string $indexType One of the class type constants TableMap::TYPE_PHPNAME, TableMap::TYPE_CAMELNAME
     *                           TableMap::TYPE_COLNAME, TableMap::TYPE_FIELDNAME, TableMap::TYPE_NUM

Parameter comment must end with a full stop

     *
     * @return mixed The primary key of the row

Consider making the return type a bit more specific; maybe use array<integer|string>.

     */
    public static function getPrimaryKeyFromRow($row, $offset = 0, $indexType = TableMap::TYPE_NUM)
    {
            $pks = [];

        $pks[] = (int) $row[

Space found after square bracket; expected "[$indexType" but found "[
$indexType"

            $indexType == TableMap::TYPE_NUM
                ? 0 + $offset
                : self::translateFieldName('Id', TableMap::TYPE_PHPNAME, $indexType)
        ];

Space found before square bracket; expected ")]" but found ")
]"

        $pks[] = (string) $row[

Space found after square bracket; expected "[$indexType" but found "[
$indexType"

            $indexType == TableMap::TYPE_NUM
                ? 1 + $offset
                : self::translateFieldName('Locale', TableMap::TYPE_PHPNAME, $indexType)
        ];

Space found before square bracket; expected ")]" but found ")
]"

        return $pks;
    }

    /**
     * The class that the tableMap will make instances of.
     *
     * If $withPrefix is true, the returned path
     * uses a dot-path notation which is translated into a path
     * relative to a location on the PHP include_path.
     * (e.g. path.to.MyClass -> 'path/to/MyClass.php')
     *
     * @param boolean $withPrefix Whether or not to return the path with the class name
     * @return string path.to.ClassName
     */
    public static function getOMClass($withPrefix = true)
    {
        return $withPrefix ? BannerImageI18nTableMap::CLASS_DEFAULT : BannerImageI18nTableMap::OM_CLASS;

As per coding style, self should be used for accessing local static members.

    }

    /**
     * Populates an object of the default type or an object that inherit from the default.
     *
     * @param array  $row       row returned by DataFetcher->fetch().

Expected 1 spaces after parameter type; 2 found

     * @param int    $offset    The 0-based offset for reading from the resultset row.

Expected 1 spaces after parameter type; 4 found

     * @param string $indexType The index type of $row. Mostly DataFetcher->getIndexType().
                                 One of the class type constants TableMap::TYPE_PHPNAME, TableMap::TYPE_CAMELNAME
     *                           TableMap::TYPE_COLNAME, TableMap::TYPE_FIELDNAME, TableMap::TYPE_NUM.
     *
     * @throws PropelException Any exceptions caught during processing will be
     *                         rethrown wrapped into a PropelException.

@throws tag comment must start with a capital letter

     * @return array           (BannerImageI18n object, last column rank)
     */
    public static function populateObject($row, $offset = 0, $indexType = TableMap::TYPE_NUM)
    {
        $key = BannerImageI18nTableMap::getPrimaryKeyHashFromRow($row, $offset, $indexType);

As per coding style, self should be used for accessing local static members.

        if (null !== ($obj = BannerImageI18nTableMap::getInstanceFromPool($key))) {

As per coding style, self should be used for accessing local static members.

            // We no longer rehydrate the object, since this can cause data loss.
            // See http://www.propelorm.org/ticket/509
            // $obj->hydrate($row, $offset, true); // rehydrate
            $col = $offset + BannerImageI18nTableMap::NUM_HYDRATE_COLUMNS;

As per coding style, self should be used for accessing local static members.

        } else {
            $cls = BannerImageI18nTableMap::OM_CLASS;

As per coding style, self should be used for accessing local static members.

            /** @var BannerImageI18n $obj */
            $obj = new $cls();
            $col = $obj->hydrate($row, $offset, false, $indexType);
            BannerImageI18nTableMap::addInstanceToPool($obj, $key);

As per coding style, self should be used for accessing local static members.

        }

        return array($obj, $col);
    }

    /**
     * The returned array will contain objects of the default type or
     * objects that inherit from the default.
     *
     * @param DataFetcherInterface $dataFetcher
     * @return array
     * @throws PropelException Any exceptions caught during processing will be
     *                         rethrown wrapped into a PropelException.

@throws tag comment must start with a capital letter

     */
    public static function populateObjects(DataFetcherInterface $dataFetcher)
    {
        $results = array();

        // set the class once to avoid overhead in the loop
        $cls = static::getOMClass(false);
        // populate the object(s)
        while ($row = $dataFetcher->fetch()) {
            $key = BannerImageI18nTableMap::getPrimaryKeyHashFromRow($row, 0, $dataFetcher->getIndexType());

As per coding style, self should be used for accessing local static members.

            if (null !== ($obj = BannerImageI18nTableMap::getInstanceFromPool($key))) {

As per coding style, self should be used for accessing local static members.

                // We no longer rehydrate the object, since this can cause data loss.
                // See http://www.propelorm.org/ticket/509
                // $obj->hydrate($row, 0, true); // rehydrate
                $results[] = $obj;
            } else {
                /** @var BannerImageI18n $obj */
                $obj = new $cls();
                $obj->hydrate($row);
                $results[] = $obj;
                BannerImageI18nTableMap::addInstanceToPool($obj, $key);

As per coding style, self should be used for accessing local static members.

            } // if key exists
        }

        return $results;
    }

Expected 1 blank line after function; 0 found

    /**
     * Add all the columns needed to create a new object.
     *
     * Note: any columns that were marked with lazyLoad="true" in the
     * XML schema will not be added to the select list and only loaded
     * on demand.
     *
     * @param Criteria $criteria object containing the columns to add.
     * @param string   $alias    optional table alias

Expected 1 spaces after parameter type; 3 found

     * @throws PropelException Any exceptions caught during processing will be
     *                         rethrown wrapped into a PropelException.

@throws tag comment must start with a capital letter

     */
    public static function addSelectColumns(Criteria $criteria, $alias = null)
    {
        if (null === $alias) {
            $criteria->addSelectColumn(BannerImageI18nTableMap::COL_ID);

As per coding style, self should be used for accessing local static members.

            $criteria->addSelectColumn(BannerImageI18nTableMap::COL_LOCALE);

As per coding style, self should be used for accessing local static members.

            $criteria->addSelectColumn(BannerImageI18nTableMap::COL_SRC);

As per coding style, self should be used for accessing local static members.

            $criteria->addSelectColumn(BannerImageI18nTableMap::COL_NAME);

As per coding style, self should be used for accessing local static members.

            $criteria->addSelectColumn(BannerImageI18nTableMap::COL_CLICKS);

As per coding style, self should be used for accessing local static members.

            $criteria->addSelectColumn(BannerImageI18nTableMap::COL_DESCRIPTION);

As per coding style, self should be used for accessing local static members.

        } else {
            $criteria->addSelectColumn($alias . '.id');
            $criteria->addSelectColumn($alias . '.locale');
            $criteria->addSelectColumn($alias . '.src');
            $criteria->addSelectColumn($alias . '.name');
            $criteria->addSelectColumn($alias . '.clicks');
            $criteria->addSelectColumn($alias . '.description');
        }
    }

    /**
     * Returns the TableMap related to this object.
     * This method is not needed for general use but a specific application could have a need.
     * @return TableMap
     * @throws PropelException Any exceptions caught during processing will be
     *                         rethrown wrapped into a PropelException.

@throws tag comment must start with a capital letter

     */
    public static function getTableMap()
    {
        return Propel::getServiceContainer()->getDatabaseMap(BannerImageI18nTableMap::DATABASE_NAME)->getTable(BannerImageI18nTableMap::TABLE_NAME);

As per coding style, self should be used for accessing local static members.

    }

    /**
     * Add a TableMap instance to the database for this tableMap class.
     */
    public static function buildTableMap()
    {
        $dbMap = Propel::getServiceContainer()->getDatabaseMap(BannerImageI18nTableMap::DATABASE_NAME);

As per coding style, self should be used for accessing local static members.

        if (!$dbMap->hasTable(BannerImageI18nTableMap::TABLE_NAME)) {

As per coding style, self should be used for accessing local static members.

            $dbMap->addTableObject(new BannerImageI18nTableMap());
        }
    }

    /**
     * Performs a DELETE on the database, given a BannerImageI18n or Criteria object OR a primary key value.
     *
     * @param mixed               $values Criteria or BannerImageI18n object or primary key or array of primary keys

Expected 1 spaces after parameter type; 15 found

     *              which is used to create the DELETE statement

Parameter comment must end with a full stop

     * @param  ConnectionInterface $con the connection to use
     * @return int             The number of affected rows (if supported by underlying database driver).  This includes CASCADE-related rows
     *                         if supported by native driver or if emulated using Propel.
     * @throws PropelException Any exceptions caught during processing will be
     *                         rethrown wrapped into a PropelException.

@throws tag comment must start with a capital letter

     */
     public static function doDelete($values, ConnectionInterface $con = null)
     {
        if (null === $con) {
            $con = Propel::getServiceContainer()->getWriteConnection(BannerImageI18nTableMap::DATABASE_NAME);

As per coding style, self should be used for accessing local static members.

        }

        if ($values instanceof Criteria) {

The class Propel\Runtime\ActiveQuery\Criteria does not exist. Did you forget a USE statement, or did you not list all dependencies?

            // rename for clarity
            $criteria = $values;
        } elseif ($values instanceof \xbanners\models\BannerImageI18n) { // it's a model object

There should be no white space after an opening "{"

            // create criteria based on pk values
            $criteria = $values->buildPkeyCriteria();
        } else { // it's a primary key, or an array of pks

There should be no white space after an opening "{"

            $criteria = new Criteria(BannerImageI18nTableMap::DATABASE_NAME);

As per coding style, self should be used for accessing local static members.

            // primary key is composite; we therefore, expect
            // the primary key passed to be an array of pkey values
            if (count($values) == count($values, COUNT_RECURSIVE)) {
                // array is not multi-dimensional
                $values = array($values);
            }
            foreach ($values as $value) {
                $criterion = $criteria->getNewCriterion(BannerImageI18nTableMap::COL_ID, $value[0]);

As per coding style, self should be used for accessing local static members.

                $criterion->addAnd($criteria->getNewCriterion(BannerImageI18nTableMap::COL_LOCALE, $value[1]));

As per coding style, self should be used for accessing local static members.

                $criteria->addOr($criterion);
            }
        }

        $query = BannerImageI18nQuery::create()->mergeWith($criteria);

        if ($values instanceof Criteria) {

The class Propel\Runtime\ActiveQuery\Criteria does not exist. Did you forget a USE statement, or did you not list all dependencies?

            BannerImageI18nTableMap::clearInstancePool();

As per coding style, self should be used for accessing local static members.

        } elseif (!is_object($values)) { // it's a primary key, or an array of pks

There should be no white space after an opening "{"

            foreach ((array) $values as $singleval) {
                BannerImageI18nTableMap::removeInstanceFromPool($singleval);

As per coding style, self should be used for accessing local static members.

            }
        }

        return $query->delete($con);
    }

Closing brace indented incorrectly; expected 5 spaces, found 4

    /**
     * Deletes all rows from the banner_image_i18n table.
     *
     * @param ConnectionInterface $con the connection to use
     * @return int The number of affected rows (if supported by underlying database driver).
     */
    public static function doDeleteAll(ConnectionInterface $con = null)
    {
        return BannerImageI18nQuery::create()->doDeleteAll($con);
    }

    /**
     * Performs an INSERT on the database, given a BannerImageI18n or Criteria object.
     *
     * @param mixed               $criteria Criteria or BannerImageI18n object containing data that is used to create the INSERT statement.

Expected 1 spaces after parameter type; 15 found

     * @param ConnectionInterface $con the ConnectionInterface connection to use
     * @return mixed           The new primary key.
     * @throws PropelException Any exceptions caught during processing will be
     *                         rethrown wrapped into a PropelException.

@throws tag comment must start with a capital letter

     */
    public static function doInsert($criteria, ConnectionInterface $con = null)
    {
        if (null === $con) {
            $con = Propel::getServiceContainer()->getWriteConnection(BannerImageI18nTableMap::DATABASE_NAME);

As per coding style, self should be used for accessing local static members.

        }

        if ($criteria instanceof Criteria) {

The class Propel\Runtime\ActiveQuery\Criteria does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $criteria = clone $criteria; // rename for clarity
        } else {
            $criteria = $criteria->buildCriteria(); // build Criteria from BannerImageI18n object
        }

Functions must not contain multiple empty lines in a row; found 2 empty lines

        // Set the correct dbName
        $query = BannerImageI18nQuery::create()->mergeWith($criteria);

        // use transaction because $criteria could contain info
        // for more than one table (I guess, conceivably)
        return $con->transaction(function () use ($con, $query) {
            return $query->doInsert($con);
        });
    }

} // BannerImageI18nTableMap
// This is the static code needed to register the TableMap for this table with the main Propel class.
//
BannerImageI18nTableMap::buildTableMap();

As per coding style, files should not end with a newline character.