Incorrect behaviour in DbManager #13501
Labels
Milestone
Comments
samdark
added
PostgreSQL
status:to be verified
|
There is no such bug in 2.0.10, so it's not db related problem. If issue will be confirmed, may this be fixed in 2.0.11.2? It looks serious. |
|
Probably. Can you find out which commit breaks it using |
|
Yep. I'll check. |
|
To fix it a check should be added that will read all data from the stream. possibly via http://php.net/manual/en/function.stream-get-contents.php |
cebe
added
status:ready for adoption
|
@cebe Like this? StalkAlex@5a76ef1 Checked in own project, it worked. I could make PR. |
|
Looks OK. |
samdark
pushed a commit
that referenced
this issue
Feb 4, 2017
…anager::getRules()` to properly handle resource data came from Rule table when using PostgreSQL
|
There is now a 2.0.11.2 release including this fix. https://github.com/yiisoft/yii2/blob/2.0.11.2/framework/CHANGELOG.md#20112-february-08-2017 |
|
Good news, thanks! |
|
This problem remain also in the 2.0.11.2 version in the method loadFromCache() in DbManager class (line 990). Could you fix it? Thank you |
|
@simonevallana it's fixed in 2.0.12 which isn't released yet. |
|
@samdark Perfect, thank you. |
Closed
|
turns out RBAC tests do not cover pgsql and sqlite, even though they suggest they should. |
cebe
added a commit
that referenced
this issue
May 31, 2017
tests were reusing static DB connection instance resulting in all tests to be run against MySQL only! PgSQL and Sqlite tests are now failing because of the issue reported in #13501.
cebe
pushed a commit
that referenced
this issue
May 31, 2017
mixartemev
added a commit
to mixartemev/yii2
that referenced
this issue
Jun 24, 2017
* Fixes yiisoft#13945: Removed Courier New from error page fonts list since it looks bad on Linux (yiisoft#13947) * Fixed phpdoc broken links to PHP manual pages [skip ci] * Fixes yiisoft#10675: Added docs for disabling CSRF validation in standalone actions * updated docs for base64UrlEncode() as discussed in yiisoft#13411 (comment) * Fix `\yii\db\QueryTrait` type hints (yiisoft#13956) [skip ci] * Fix `\yii\db\QueryInterface` type hints (yiisoft#13959) [skip ci] * Added missing parts of disabling CSRF validation doc [skip ci] (yiisoft#13966) * Fixed wording in Russian translation (yiisoft#13970) [skip ci] * Fixes yiisoft#13961: RBAC Rules: PostgreSQL: PHP Warning "unserialize() expects parameter 1 to be string, resource given" was fixed * compatibility for PHPUnit adjusted * compatibility with PHPUnit 6.x added * unit test fix * `yii\caching\Cache::getOrSet()` now supports both `Closure` and `callable` Closes yiisoft#13981 * compatibility with PHPUnit 6.x fix * garbage collection added to unit tests * compatibility with PHPUnit 6.x fix * fixed tests without assettions * compatibility with PHPUnit 6.x fix * compatibility with PHPUnit 6.x fix * memory usage at unit test reduced * update phpdoc annotation in BaseActiveRecord yiisoft/yii2-gii#267 (comment) * added upgrade note about yii\cache\Cache::getOrSet() * Revert "update phpdoc annotation in BaseActiveRecord" This reverts commit 0f78008. yiisoft/yii2-gii#267 (comment) * Tweaked verbiage slightly [ci skip] * yiisoft#13975: make documentation of current behaviour more clear [skip ci] * docs/guide/input-multiple-models: added a note on avoiding validating the same input data with the same rules more than once (yiisoft#13995) [skip ci] * Fixes yiisoft#13807: Fixed `yii\db\QueryBuilder` to inherit subquery params when building a `INSERT INTO ... SELECT` query * Tiny documentation fix [skip ci] (yiisoft#13998) * Added note to findOne and findAll docs (yiisoft#14002) [skip ci] Added note to findOne and findAll docs to clearly state that complex conditions are not supported. * [IpValidator] disable ipv6 checks (yiisoft#13984) * disable ipv6 checks * remove empty line * updated CHANGELOG.md * Added AF_INET6 to requirements * Enhanced requirements memo for IPv6 * update AR docs * Update composer fxp plugin version in docs and travis scripts * Fixes yiisoft#13963: Added tests for yii\behaviors\TimestampBehavior * Fixes yiisoft#13911: Significantly enhanced MSSQL schema reading performance * Skipped segfaulting test when running on Travis and corresponding PHP versions * enforce `backupGlobals` enabled for PHPUnit * Fix `TimestampBehaviorTest` to be compatible with PHPUnit 6 * Switched to asset-packagist * Enable xdebug for coverage on 7.1 insted of 5.6 as it should work faster * Switched ocular to use PHP 7.1 execution results * Added note clarifying that load function clears fixture data [skip ci] * Fixes yiisoft#13728: Fixed the bug when `yii\behaviors\SluggableBehavior` wasn't preserving immutable slug values * Changed return value in MemCache::setValues() * Fixes yiisoft#13362: Fixed return value of * Fixes yiisoft#14012: `yii\db\pgsql\Schema::findViewNames()` was skipping materialized views * Created issue template * Wrong repo [skip ci] * Fixes yiisoft#13694: yii\widgets\Pjax now sends X-Pjax-Url header with response to fix redirect * Fixed PHP manual link language [skip ci] * Add setup for Apache Describes the settings for Apache. * yii\filters\RateLimiter refactored (yiisoft#13994) Refactored yii\filters\RateLimiter and added tests * Fixed spelling and typos * Updated slovak localization Fixed missing translation for '{attribute} must be an integer.' string * Fixed ignored params when using count in `SqlDataProvider` * Fixed ambiguous error sql while using unique validator * Refactored solution to be contained within ActiveQuery instead of helper * Removed unneeded changelog line [skip ci] * Removed obsolete import * improve session PHPdoc * Applied disambiguation to ExistValidator as well * Fixes yiisoft#10346: Fixed "DOMException: Invalid Character Error" in `yii\web\XmlResponseFormatter::buildXml()` * Fixes yiisoft#13087: Fixed getting active validators for safe attribute * Added `yii\data\Sort::parseSortParams` allowing customize other params request formats in descendant class. close yiisoft#13170 * Fixes yiisoft#11719: Fixed `yii\db\Connection::$enableQueryCache` caused infinite loop when the same connection was used for `yii\caching\DbCache` * Adjusted code style for less conditions branching * Fixes yiisoft#13226: `yii cache` command now warns about the fact that it's not able to flush APC cache from console * Fixes yiisoft#13848: `yii\di\Instance::ensure()` wasn't throwing an exception when `$type` is specified and `$reference` object isn't instance of `$type` * Fixes yiisoft#13689: Fixed handling of errors in closures * Fixed changelog [skip ci] * PHPDocs and code style fixed * Helper independent (yiisoft#14050) * Made sure most helpers are independent of Yii::$app * Made sure most validators are independent of Yii::$app * Fixes yiisoft#14033: Fixed `yii\filters\AccessRule::matchIp()` erroring in case IP is not defined under HHVM * Removed redundant use * Removed redundant else * param → parameter [skip ci] * Adjusted changelog [skip ci] * Update runtime-routing.md * Fixes yiisoft#13790: Fixed error in `\yii\widgets\MaskedInput` JavaScript by raising version required * Fixes yiisoft#14052: Fixed processing parse errors on PHP 7 since these are instances of `\ParseError` * Fixes yiisoft#13951: renamed get-method which was conflicting with existing property (yiisoft#14055) * Alternative names * Fixes yiisoft#14059: Removed unused AR instantiating for calling of static methods * Add phpdoc. [skip ci] * Fix an invalid phpDocumentor annotation Fixed an invalid phpDocumentor annotation of `yii\data\DataProviderInterface::getPagination()`. * Fixes yiisoft#14072: clear table schema cache for all methods that affect cache * Reorder changelog * file will not loaded https://github.com/yiisoft/yii2/tree/master/framework/test * Fix validators documentation * More control over ActiveForm widget If part of a form is rendered separately (e.g. in ajax request), it's not easy to apply JS validation to it. By moving JS registration to a separate method this problem could be solved much more easier. * Renamed `ActiveForm::registerJs()` to `registerClientScript()`, added tests * Fixed tests * Fixed CHANGELOG * Reordered CHANGELOG * fixed missing changelog and missing `@since` * Skip testAutoRefreshTableSchema on sqllite * Skip just certain things for sqllite * !== * Update phpdoc, \yii\db\ActiveRecord => \yii\db\ActiveRecordInterface [skip ci] (yiisoft#14076) * Cleanup after yiisoft#14017 (yiisoft#14091) * Fixed Oracle SQL queries with `IN` condition and more than 1000 parameters * Updated PHPdoc * Remove unused imports. (yiisoft#14096) * Fixes yiisoft#4793: `yii\filters\AccessControl` now can be used without `user` component * Fixes yiisoft#14098: `yii\helpers\BaseFileHelper::normalizeOptions()` is now protected * Fixes yiisoft#14108: fixed phpdoc mistake in method description * Update db-migrations.md (yiisoft#14109) [skip ci] By default, the class BaseMigrateController property migrationPath is assigned the value '@app/migrations'. In the method getNewMigrations() is the conversion migrationNamescpaces in the path to the file and merges it all into a single array along with the path migrationPath. If the project for migration module is a separate table in the database, all migration came from migrationPath are not initialized and therefore try to migrate. Therefore, it is necessary to specify the property 'migrationPath' => null. * Eliminated else branches in beforeSave() and beforeDelete() phpdoc examples [skip ci] * Fixes yiisoft#13890: DbTarget log transaction bug * Update db-migrations.md (yiisoft#14113) [skip ci] I made a typo. Sorry. The command migration-rbac, migrationPath property already exists. * Typo (yiisoft#14117) [skip ci] * Add Indonesian guide translation (yiisoft#14120) [skip ci] * Update README.md on Indonesian Guide Translation * Translate "Working with Forms" guide to Bahasa Indonesia * added option to disable query logging in DB command fixes yiisoft#12528 * moved query log logic into private method and reduced calls to getRawSql() fixes yiisoft#12528 * failing test for asset bundle publishing empty directories * Added copyEmptyDirectories option to FileHelper also set it to false in AssetManager to avoid creating a lot of empty directories. fixes yiisoft#9669 * made AssetBundle tests less cryptic cleanup step was added in setUp() so this will still work. split test functions filehelper test * Fixes yiisoft#5442: Fixed problem on load fixture dependencies with database related tests * Fixed changelog [skip ci] * Added support for sorting by expression to `\yii\data\Sort` * function clear() -- Set populated arrays to empty arrays (yiisoft#14067) `yii\web\View::clear()` sets populated arrays to empty arrays * changelog for yiisoft#14067 * Further changes for unique and exist validators - Combined methods for getting names and aliases of from tables - Normalized names and aliases - Added MSSQL syntax - Added support for spaces in aliases and table names * added changelog for yiisoft#12528 * Do not render empty row in GridView when data is empty and emptyText set to false Fixes yiisoft#13352 close yiisoft#13422 * Added support for cloning a db connection improved fix yiisoft#14020 fixes yiisoft#13890 https://github.com/yiisoft/yii2/pull/14020/files#r115185865 close yiisoft#14121 * Fixed ambiguous column name in SELECT in UniqueValidator fixes yiisoft#14042 * Added escaping for column name * Fixes yiisoft#13350, yiisoft#14094, yiisoft#11288 - Bug yiisoft#13350: Fixed bug with incorrect caching of `yii\web\UrlRule::createUrl()` results in `yii\web\UrlManager`. - Bug yiisoft#14094: Fixed bug when single `yii\web\UrlManager::createUrl()` call my result multiple calls of `yii\web\UrlRule::createUrl()` for the same rule. - Enh yiisoft#11288: Added support for caching of `yii\web\UrlRule::createUrl()` results in `yii\web\UrlManager` for rules with defaults. * Remove unnecessary variable from UrlRule.php (yiisoft#13948) * Additional refactoring as suggested by @dynasource * fix broken link (yiisoft#14143) [skip ci] File output-client-scripts.md not created. And in site - error 404: http://stuff.cebe.cc/yii2docs-ru/guide-output-client-scripts.html * "из вне" => "извне" grammar nazi attack * Fixed changelog [skip ci] * Fixes yiisoft#14133: Fixed bug when calculating timings with mixed nested profile begin and end in `yii\log\Logger::calculateTimings()` * Fixes yiisoft#14150 * Translate "Working with Database" guide to Bahasa Indonesia (yiisoft#14152) [skip ci] * yiisoft#14150: Added {{ and }} to getTablesUsedInFrom() output * Additional edge case for yiisoft#14150 * Translate "Generating Code with Gii" guide to Bahasa Indonesia (yiisoft#14162) * Fixes yiisoft#14161, yiisoft#14150: fixed primary table aliasing (include case manual set alias) in validators * added namespace example to the autoload guide * Translate "Looking Ahead" guide to Bahasa Indonesia (yiisoft#14177) * advanced example on filter validator fixes yiisoft#4284 * Simplified Chinese translation of filter validator (yiisoft#14195) * Simplified Chinese Translation of filter validator * Using more specific words * Fixed typo in BaseFileHelper::findFiles() docs (yiisoft#14200) [skip ci] * Add test coverage of yii\helpers\BaseArrayHelper (yiisoft#14205) * Translate some application structure guide to Bahasa Indonesia (yiisoft#14209) [skip ci] * Add blocktypes.json file in Indonesian Guide * Translate "Application Structure Overview" to Bahasa Indonesia * Translate "Entry Script" guide to Bahasa Indonesia * Translate "Structure App - Application" to Bahasa Indonesia * Translate "Application Component" guide to Bahasa Indonesia * Add translators.json attribution in Indonesian Documentation * Fixes yiisoft#14211: Fixed regression in Unique and Exist validators * fixed indonesian translators.json file * add doc for issue yiisoft#8420 yiisoft#8420 * typo * Adjusted text [skip ci] * MySql -> MySQL typo * Adjusted typo text [skip ci] * Fixes yiisoft#13846: Fixed `Query::count()` issue with `orderBy` * Inserted missing * on line 972 (yiisoft#14224) [skip ci] * Add test coverage of yii\helpers\BaseMarkdown (yiisoft#14223) * Add test coverage of yii\helpers\BaseJson (yiisoft#14217) * Adjusted text [skip ci] * Adjusted text [skip ci] * Adjusted text [skip ci] * Adjusted text [skip ci] * Add test coverage of yii\helpers\Html (yiisoft#14220) * Add migration file example for DbSession (yiisoft#14227) [skip ci] Creating a `char` primaryKey in a migration is non-obvious. Used solution from this [issue comment](yiisoft#10889 (comment)). * Adjusted text [skip ci] * Fixed yiisoft#13551: loading fixtures in subdirectories * Enhanced FixtureController::getFixtureRelativeName() Updated CHANGELOG * Adjusted text，add some adivse [skip ci] * Alternative implementation for passing parameters to AccessRule roles fixes yiisoft#8426: `yii\filters\AccessRule` now allows passing GET or other parameters to the role checking function replaces yiisoft#8426 * Document queryParams in the authorization guide * simplified guide example * improved docs * Fixed RBAC databases tests tests were reusing static DB connection instance resulting in all tests to be run against MySQL only! PgSQL and Sqlite tests are now failing because of the issue reported in yiisoft#13501. * added failing test for yiisoft#13593 * additional fix for yiisoft#13501 close yiisoft#13593 * fix cloning of DB connection for sqlite in-memory db fixes yiisoft#14131 close yiisoft#14232 * Allows override query relation in descendant class fixes yiisoft#13441 close yiisoft#13642 * Fixes yiisoft#13058: Fixed caught exception thrown during view file rendering produces wrong output * improved docs: link term "path alias" to the guide * Added support for specifying aliases for migrationNamespaces This is used to specify pathes to migrations that do not have namespaces. While not directly supported by the migration command provideded by the framework, these migrations exist in a lot of extensions because custom implementations of migration controllers out of the framework were using this approach to load multiple migrations from multiple paths. Even the framework itself currently ships non-namespaced migrations: - https://github.com/yiisoft/yii2/blob/17a1d91e4a517f4f15dce973bf3c50dd939dce63/framework/rbac/migrations/m140506_102106_rbac_init.php - https://github.com/yiisoft/yii2/blob/17a1d91e4a517f4f15dce973bf3c50dd939dce63/framework/caching/migrations/m150909_153426_cache_init.php - https://github.com/yiisoft/yii2/blob/17a1d91e4a517f4f15dce973bf3c50dd939dce63/framework/log/migrations/m141106_185632_log_init.php This change allows existing applications to adopt the new namespace-based approach while keeping existing migrations. While it would be possible to add namespaces to migrations in the application itself, it is not easily possible to add namespaces to migrations that come from external sources like extensions. * added alias syntax support fixes yiisoft#13356 fixes yiisoft#13359 * make migrationPath accept an array * moved include file logic to separate method * Fixes yiisoft#4999: Added support for wildcards at `yii\filters\AccessRule::$controllers` * Update BaseMigrateController.php make sure console arguments are recognized as array. * Fixes yiisoft#5108 `DateValidator` resets `$timestampAttribute` value on empty attribute (yiisoft#14242) * `yii\validators\DateValidator` now resets `$timestampAttribute` value on empty validated attribute value * array-value test at `DateValidatorTest` restored * skip failing HHVM test see yiisoft#14244 * moved wrong changelog entry * fixed version and missing `@since` annotation * release version 2.0.12 * prepare for next release * Greek translation update. * Fixed Formatter::normalizeDatetimeValue() docs There was a reference to `$checkTimeInfo` that was leftover from 54278fc * improved formatter documentation * Fix MessageController color output on non-color termnials fixes yiisoft#14248 close yiisoft#14249 * guide: note about param binding on strings, not arrays issue yiisoft#14218 * added param binding examples for update() and delete() fixes yiisoft#14218 * Fixes yiisoft#14264: Fixed a bug where `yii\log\Logger::calculateTimings()` was not accepting messages with array tokens * Improved base migration controller documentation [skip ci] (yiisoft#14253) * added example for using unsupported PHP format in yii formatter fixes yiisoft#14278 * Fixes yiisoft#14201: `yii\console\controllers\MessageController::extractMessagesFromTokens()` is now protected * Fixes yiisoft#13787: Added `yii\db\Migration::$maxSqlOutputLength` that allows limiting number of characters for outputting SQL * Adjusted RBAC docs to offer both migrations and command approaches [skip ci] * Fixes yiisoft#14089: Added tests for `yii\base\Theme` * Fixes yiisoft#13586: Added `$preserveNonEmptyValues` property to the `yii\behaviors\AttributeBehavior` * Fixes yiisoft#14192: Fixed wrong default null value for TIMESTAMP when using PostgreSQL * Fixes yiisoft#14081: Added `yii\caching\CacheInterface` to make custom cache extensions adoption easier * Removed unused local var in JS test (yiisoft#14093) * Added php-cs-fixer coding standards validation to Travis CI (yiisoft#14100) * php-cs-fixer: PSR2 rule. * php-cs-fixer: PSR2 rule - fix views. * Travis setup refactoring. * Add php-cs-fixer to travis cs tests. * Fix tests on hhvm-3.12 * improve travis config * composer update * revert composer update * improve travis config * Fix CS. * Extract config to separate classes. * Extract config to separate classes. * Add file header. * Force short array syntax. * binary_operator_spaces fixer * Fix broken tests * cast_spaces fixer * concat_space fixer * dir_constant fixer * ereg_to_preg fixer * function_typehint_space fixer * hash_to_slash_comment fixer * is_null fixer * linebreak_after_opening_tag fixer * lowercase_cast fixer * magic_constant_casing fixer * modernize_types_casting fixer * native_function_casing fixer * new_with_braces fixer * no_alias_functions fixer * no_blank_lines_after_class_opening fixer * no_blank_lines_after_phpdoc fixer * no_empty_comment fixer * no_empty_phpdoc fixer * no_empty_statement fixer * no_extra_consecutive_blank_lines fixer * no_leading_import_slash fixer * no_leading_namespace_whitespace fixer * no_mixed_echo_print fixer * no_multiline_whitespace_around_double_arrow fixer * no_multiline_whitespace_before_semicolons fixer * no_php4_constructor fixer * no_short_bool_cast fixer * no_singleline_whitespace_before_semicolons fixer * no_spaces_around_offset fixer * no_trailing_comma_in_list_call fixer * no_trailing_comma_in_singleline_array fixer * no_unneeded_control_parentheses fixer * no_unused_imports fixer * no_useless_return fixer * no_whitespace_before_comma_in_array fixer * no_whitespace_in_blank_line fixer * not_operator_with_successor_space fixer * object_operator_without_whitespace fixer * ordered_imports fixer * php_unit_construct fixer * php_unit_dedicate_assert fixer * php_unit_fqcn_annotation fixer * phpdoc_indent fixer * phpdoc_no_access fixer * phpdoc_no_empty_return fixer * phpdoc_no_package fixer * phpdoc_no_useless_inheritdoc fixer * Fix broken tests * phpdoc_return_self_reference fixer * phpdoc_single_line_var_spacing fixer * phpdoc_single_line_var_spacing fixer * phpdoc_to_comment fixer * phpdoc_trim fixer * phpdoc_var_without_name fixer * psr4 fixer * self_accessor fixer * short_scalar_cast fixer * single_blank_line_before_namespace fixer * single_quote fixer * standardize_not_equals fixer * ternary_operator_spaces fixer * trailing_comma_in_multiline_array fixer * trim_array_spaces fixer * protected_to_private fixer * unary_operator_spaces fixer * whitespace_after_comma_in_array fixer * `parent::setRules()` -> `$this->setRules()` * blank_line_after_opening_tag fixer * Update finder config. * Revert changes for YiiRequirementChecker. * Fix array formatting. * Add missing import. * Fix CS for new code merged from master. * Fix some indentation issues. * ensure tablenames are not replaced in insert and update * Fixes yiisoft#14286: Used primary inputmask package name instead of an alias * Documented possibility of using asset-packagist * fixed typo * Fixes yiisoft#14307: Fixed PHP warning when `yii\console\UnknownCommandException` is thrown for empty command * Fixed phpdoc `yii\test\ActiveFixture::load` (yiisoft#14292) [skip ci] * Fixes yiisoft#13824: Support extracting concatenated strings in `yii message` * Fixes yiisoft#14298: The default response formatter configs defined by `yii\web\Response::defaultFormatters()` now use the array syntax * Added link to Russian Sphinx guide [skip ci] * Removed cs fixer from Travis * Fixed typo [skip ci] * Fixes yiisoft#14341: Fixed regression in error handling introduced by fixing yiisoft#14264
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What steps will reproduce the problem?
Trying to create permission
data of rule is
O:24:"app\rules\TestRule":3:{s:4:"name";s:14:"TestOnly";s:9:"createdAt";i:1485582849;s:9:"updatedAt";i:1485582849;}What do you get instead?
Error
Here's screen. As it seen data variable isn't sting anymore but resource stream.
Additional info
The text was updated successfully, but these errors were encountered: