umbrellio / laravel-pg-extensions

100%

master: 100%

LAST BUILD ON BRANCH 2.5.1

branch: 2.5.1

CHANGE BRANCH

x

Reset

• 2.5.1
• master
• refactor-blueprint
• 2.0.0
• feature/views
• feature/unique-wheres
• 2.1.0
• feature/tsrange
• feature/default-expression
• feature/using
• feature/extensions-support
• feature/exclude-gist
• feature/has-index
• 2.2.0
• bug/fix-access
• 2.2.1
• bug/connections
• 2.2.2
• 2.2.3
• bug/postgres-connection
• 2.2.2.1
• 2.2.2.2
• 2.2.4
• 2.2.5
• 2.2.6
• doctrine/refactor
• test-travis
• add_numeric_column_type
• 2.3.0
• 2.4.0
• 2.5.0
• bugs/unique-constraints
• feature/exclude-constraint
• 2.6.0
• fix-traits-namespace
• 2.6.1
• 2.7.0
• feature/refactor-extensionable
• 3.0.0
• system/upgrade-dependencies
• 4.0.0
• add_ATTR_EMULATE_PREPARES_support
• 4.1.0
• feature/fix_get_pdo
• fix/pdo_field
• 4.1.1
• extensions/initialize
• 4.1.2
• 4.2.0
• laravel_8_support
• 4.3.0
• upgrade/php7.4
• feature/triggers
• types/tsrange
• 4.4.0
• bug/extension
• 4.4.1
• bug/connection
• build/migrate-github-actions
• 4.5.2
• build/workflow
• minor/badges
• fix/composer
• upgrade/php8
• build/release
• build/test-release
• bug/tsrange
• tests

Build # 306

Build Type

push

travis-ci

Committed by lazeevv

Commit Message

Added support deleting UNIQUE partial indexes (#22)

Added support for deleting unique indexes with WHERE conditions.

By default, if the created unique index contains no conditions, PostgresSQL automatically creates a Constraint for it, for example:

CREATE UNIQUE INDEX CONCURRENTLY examples_new_col_idx ON examples (new_col);
ALTER TABLE examples
    ADD CONSTRAINT examples_unique_constraint USING INDEX examples_new_col_idx;
If the unique index contains WHERE conditions, then such a Constraint will not be created, because PostgreSQL doesn't
define a partial (ie conditional) UNIQUE constraint.

When trying to delete such a partial unique index, we get the error "Unique Constraint not found", so the standard command to remove $table->dropUnique() does not fit, the command $table->dropIndex() is suitable, but if pass the array there, then the index name will be generated as for a regular index with the suffix _index, instead of _unique, which we need.

And manually generating the index name is wrong.

This Merge Request solves these problems.

Run Details

1 of 1 new or added line in 1 file covered. (100.0%)

253 of 253 relevant lines covered (100.0%)

70.53 hits per line