-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixed issue #3237 #3239
Closed
Closed
fixed issue #3237 #3239
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
Guite
commented
Dec 3, 2016
Q | A |
---|---|
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Fixed tickets | #3237 |
Refs tickets | - |
License | MIT |
Changelog updated | yes |
fixing #1486 for 1.3.6 on release-1.3 branch
Adding old hacks (not included into 1.3.6) into release-1.3 branch
replace use of serialize functions with json
Added parameter to retrieve an array
| Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | - | Fixed tickets | - | Refs tickets | #666 | License | MIT | Doc PR | - This fixes the problem that hte login window in IE<10 is broken when you switch on generating template comments into the output. The issue is described clearly in the #666 ticket.
| Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #704 | License | MIT | Doc PR |
| Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #924 | License | MIT | Doc PR | see also https://gist.github.com/cmfcmf/6329308
| Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #783 | License | MIT | Doc PR |
| Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #1561 | License | MIT | Doc PR |
| Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | | License | MIT | Doc PR | do not know the tickets of this fixes
This PR was merged into zikula:release-1.3 branch. Discussion ---------- [release-1.3] Refers to #666 fix for IE<10 login problems | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | - | Fixed tickets | - | Refs tickets | #666 | License | MIT | Doc PR | - This fixes the problem that hte login window in IE<10 is broken when you switch on generating template comments into the output. The issue is described clearly in the #666 ticket. Commits ------- 1346510 [release-1.3] Refers to #666 fix for IE<10 login problems | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | - | Fixed tickets | - | Refs tickets | #666 | License | MIT | Doc PR | - This fixes the problem that hte login window in IE<10 is broken when you switch on generating template comments into the output. The issue is described clearly in the #666 ticket. espaan
``|safetext`` added
``|safetext`` added
``public `` added thanks to cmf!
Korinthen for cmf ;-)
This PR was merged into zikula:release-1.3 branch. Discussion ---------- adding my 1.3.6patch repro | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | | License | MIT | Doc PR | this will close my collection at https://github.com/rallek/1.3.6patch Commits ------- be16503 workflow fix | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #704 | License | MIT | Doc PR | rallek 3d081c9 drag and drop fix for extmenu | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #924 | License | MIT | Doc PR | see also https://gist.github.com/cmfcmf/6329308 rallek 6715748 Fixed Block filtering in 1.3.5/6 | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #516, #938 | License | MIT | Doc PR | rallek 2ca1a5b Fixed displaying of selection boxes | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #762 | Refs tickets | #775 | License | MIT | Doc PR | rallek 64f0cd8 added jpeg | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #783 | License | MIT | Doc PR | rallek a2b3e3a Avoid flush by category selector | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | #1561 | License | MIT | Doc PR | rallek dd0036b some more fixes | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | Refs tickets | | License | MIT | Doc PR | do not know the tickets of this fixes rallek 2f519ce Update users_user_login.tpl ``|safetext`` added rallek 966e29b Update users_block_login.tpl ``|safetext`` added rallek b05529e Update CategorySelector.php ``public `` added thanks to cmf! rallek 20383d0 Update PageUtil.php Korinthen for cmf ;-) rallek
| Q | A | ------------- | --- | Bug fix? | yes | New feature? | no, don't think so | BC breaks? | no | Deprecations? | no | Tests pass? | Tested several modules in 1.3.6 that use DBUtil | Fixed tickets | - | Refs tickets | https://github.com/rallek/1.3.6patch/issues/8, #1603 | License | MIT | Doc PR | - This updates the DBUtil in the 1.3 branch to reflect all the updates @rgasch did to DBUtil in the 1.4 branch. Essentially a backport of the updates done in the 1.4 branch to the 1.3 branch. It's not a one2one copy since some calls are different in 1.4.0 vs 1.3.x These updates are needed for zwebstore 1.3.2 for instance and do solve several bugs. I have tested a bit with News 3.1, Content 4.1 and ZWebstore 1.3.2 that all use DBUtil. That is all ok. If people can test this DBUtil, please do. It should work. See the list below for the diff between 1.4 and 1.3, left < is 1.3 an right > 1.4 ``` 16a17,18 > * > * @deprecated 45c47 < self::$cache_enabled = ServiceUtil::getManager()->getArgument('dbcache.enable'); --- > self::$cache_enabled = ServiceUtil::getManager()->getParameter('dbcache.enable'); 63c65 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 66c68 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 87c89 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 90c92 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 105c107 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 108c110 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 188c190 < return ServiceUtil::getManager()->getArgument('dbtables'); --- > return ServiceUtil::getManager()->getParameter('dbtables'); 283c285 < //$stmt->setHydrationMode(Doctrine::HYDRATE_RECORD); --- > //$stmt->setHydrationMode(Doctrine_Core::HYDRATE_RECORD); 301,305c303,304 < if (System::isLegacyMode()) { < return new Zikula_Adapter_AdodbStatement($result); < } else { < return $result; < } --- > > return $result; 999c998 < ModUtil::available('Categories')) { --- > ModUtil::available('ZikulaCategoriesModule')) { 1207a1207,1210 > if ($table == 'categories_mapobj') { > // table no longer exists >= 1.3.6 > return true; > } 1242c1245 < ModUtil::available('Categories')) { --- > ModUtil::available('ZikulaCategoriesModule')) { 1545c1548 < $resultRows = $result->fetchAll(Doctrine::FETCH_NUM); --- > $resultRows = $result->fetchAll(Doctrine_Core::FETCH_NUM); 1595c1598 < $resultRows = $result->fetchAll(Doctrine::FETCH_ASSOC); --- > $resultRows = $result->fetchAll(Doctrine_Core::FETCH_ASSOC); 1945c1948 < if (!ModUtil::dbInfoLoad('Categories')) { --- > if (!ModUtil::dbInfoLoad('ZikulaCategoriesModule')) { 2382c2385 < $res = $res->fetchAll(Doctrine::FETCH_COLUMN); // RNG: Should this really be fetchAll() ?? --- > $res = $res->fetchAll(Doctrine_Core::FETCH_COLUMN); // RNG: Should this really be fetchAll() ?? 2782c2785 < $res = $res->fetchAll(Doctrine::FETCH_COLUMN); --- > $res = $res->fetchAll(Doctrine_Core::FETCH_COLUMN); 2904c2907 < if (($enableAllServices || (isset($tables["{$table}_db_extra_enable_categorization"]) && $tables["{$table}_db_extra_enable_categorization"])) && System::getVar('Z_CONFIG_USE_OBJECT_CATEGORIZATION') && strcmp($table, 'categories_') !== 0 && strcmp($table, 'objectdata_attributes') !== 0 && strcmp($table, 'objectdata_log') !== 0 && ModUtil::available('Categories')) { --- > if (($enableAllServices || (isset($tables["{$table}_db_extra_enable_categorization"]) && $tables["{$table}_db_extra_enable_categorization"])) && System::getVar('Z_CONFIG_USE_OBJECT_CATEGORIZATION') && strcmp($table, 'categories_') !== 0 && strcmp($table, 'objectdata_attributes') !== 0 && strcmp($table, 'objectdata_log') !== 0 && ModUtil::available('ZikulaCategoriesModule')) { 3914c3917 < return z_exit(__f('%1$s: unable to limit tablename [%2$s] because database prefix is too long for Oracle, please shorten it (recommended length is 4 chars)', array( --- > throw new \Exception(__f('%1$s: unable to limit tablename [%2$s] because database prefix is too long for Oracle, please shorten it (recommended length is 4 chars)', array( 4006c4009 < } --- > } \ No newline at end of file ```
This PR was merged into zikula:1.3 branch. Discussion ---------- Backport DBUtil fixes from 1.4 to 1.3 | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no, don't think so | BC breaks? | no | Deprecations? | no | Tests pass? | Tested several modules in 1.3.6 that use DBUtil | Fixed tickets | - | Refs tickets | https://github.com/rallek/1.3.6patch/issues/8, #1603 | License | MIT | Doc PR | - This updates the DBUtil in the 1.3 branch to reflect all the updates @rgasch did to DBUtil in the 1.4 branch. Essentially a backport of the updates done in the 1.4 branch to the 1.3 branch. It's not a one2one copy since some calls are different in 1.4.0 vs 1.3.x These updates are needed for zwebstore 1.3.2 for instance and do solve several bugs. I have tested a bit with News 3.1, Content 4.1 and ZWebstore 1.3.2 that all use DBUtil. That is all ok. If people can test this DBUtil, please do. It should work. See the list below for the diff between 1.4 and 1.3, left < is 1.3 an right > 1.4 ``` 16a17,18 > * > * @deprecated 45c47 < self::$cache_enabled = ServiceUtil::getManager()->getArgument('dbcache.enable'); --- > self::$cache_enabled = ServiceUtil::getManager()->getParameter('dbcache.enable'); 63c65 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 66c68 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 87c89 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 90c92 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 105c107 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 108c110 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 188c190 < return ServiceUtil::getManager()->getArgument('dbtables'); --- > return ServiceUtil::getManager()->getParameter('dbtables'); 283c285 < //$stmt->setHydrationMode(Doctrine::HYDRATE_RECORD); --- > //$stmt->setHydrationMode(Doctrine_Core::HYDRATE_RECORD); 301,305c303,304 < if (System::isLegacyMode()) { < return new Zikula_Adapter_AdodbStatement($result); < } else { < return $result; < } --- > > return $result; 999c998 < ModUtil::available('Categories')) { --- > ModUtil::available('ZikulaCategoriesModule')) { 1207a1207,1210 > if ($table == 'categories_mapobj') { > // table no longer exists >= 1.3.6 > return true; > } 1242c1245 < ModUtil::available('Categories')) { --- > ModUtil::available('ZikulaCategoriesModule')) { 1545c1548 < $resultRows = $result->fetchAll(Doctrine::FETCH_NUM); --- > $resultRows = $result->fetchAll(Doctrine_Core::FETCH_NUM); 1595c1598 < $resultRows = $result->fetchAll(Doctrine::FETCH_ASSOC); --- > $resultRows = $result->fetchAll(Doctrine_Core::FETCH_ASSOC); 1945c1948 < if (!ModUtil::dbInfoLoad('Categories')) { --- > if (!ModUtil::dbInfoLoad('ZikulaCategoriesModule')) { 2382c2385 < $res = $res->fetchAll(Doctrine::FETCH_COLUMN); // RNG: Should this really be fetchAll() ?? --- > $res = $res->fetchAll(Doctrine_Core::FETCH_COLUMN); // RNG: Should this really be fetchAll() ?? 2782c2785 < $res = $res->fetchAll(Doctrine::FETCH_COLUMN); --- > $res = $res->fetchAll(Doctrine_Core::FETCH_COLUMN); 2904c2907 < if (($enableAllServices || (isset($tables["{$table}_db_extra_enable_categorization"]) && $tables["{$table}_db_extra_enable_categorization"])) && System::getVar('Z_CONFIG_USE_OBJECT_CATEGORIZATION') && strcmp($table, 'categories_') !== 0 && strcmp($table, 'objectdata_attributes') !== 0 && strcmp($table, 'objectdata_log') !== 0 && ModUtil::available('Categories')) { --- > if (($enableAllServices || (isset($tables["{$table}_db_extra_enable_categorization"]) && $tables["{$table}_db_extra_enable_categorization"])) && System::getVar('Z_CONFIG_USE_OBJECT_CATEGORIZATION') && strcmp($table, 'categories_') !== 0 && strcmp($table, 'objectdata_attributes') !== 0 && strcmp($table, 'objectdata_log') !== 0 && ModUtil::available('ZikulaCategoriesModule')) { 3914c3917 < return z_exit(__f('%1$s: unable to limit tablename [%2$s] because database prefix is too long for Oracle, please shorten it (recommended length is 4 chars)', array( --- > throw new \Exception(__f('%1$s: unable to limit tablename [%2$s] because database prefix is too long for Oracle, please shorten it (recommended length is 4 chars)', array( 4006c4009 < } --- > } \ No newline at end of file ``` Commits ------- b94c920 Backport DBUtil fixes from 1.4 to 1.3 | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no, don't think so | BC breaks? | no | Deprecations? | no | Tests pass? | Tested several modules in 1.3.6 that use DBUtil | Fixed tickets | - | Refs tickets | https://github.com/rallek/1.3.6patch/issues/8, #1603 | License | MIT | Doc PR | - This updates the DBUtil in the 1.3 branch to reflect all the updates @rgasch did to DBUtil in the 1.4 branch. Essentially a backport of the updates done in the 1.4 branch to the 1.3 branch. It's not a one2one copy since some calls are different in 1.4.0 vs 1.3.x These updates are needed for zwebstore 1.3.2 for instance and do solve several bugs. I have tested a bit with News 3.1, Content 4.1 and ZWebstore 1.3.2 that all use DBUtil. That is all ok. If people can test this DBUtil, please do. It should work. See the list below for the diff between 1.4 and 1.3, left < is 1.3 an right > 1.4 ``` 16a17,18 > * > * @deprecated 45c47 < self::$cache_enabled = ServiceUtil::getManager()->getArgument('dbcache.enable'); --- > self::$cache_enabled = ServiceUtil::getManager()->getParameter('dbcache.enable'); 63c65 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 66c68 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 87c89 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 90c92 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 105c107 < $databases = ServiceUtil::getManager()->getArgument('databases'); --- > $databases = ServiceUtil::getManager()->getParameter('databases'); 108c110 < $cacheDriver = ServiceUtil::getManager()->getService('doctrine.cachedriver'); --- > $cacheDriver = ServiceUtil::getManager()->get('doctrine.cachedriver'); 188c190 < return ServiceUtil::getManager()->getArgument('dbtables'); --- > return ServiceUtil::getManager()->getParameter('dbtables'); 283c285 < //$stmt->setHydrationMode(Doctrine::HYDRATE_RECORD); --- > //$stmt->setHydrationMode(Doctrine_Core::HYDRATE_RECORD); 301,305c303,304 < if (System::isLegacyMode()) { < return new Zikula_Adapter_AdodbStatement($result); < } else { < return $result; < } --- > > return $result; 999c998 < ModUtil::available('Categories')) { --- > ModUtil::available('ZikulaCategoriesModule')) { 1207a1207,1210 > if ($table == 'categories_mapobj') { > // table no longer exists >= 1.3.6 > return true; > } 1242c1245 < ModUtil::available('Categories')) { --- > ModUtil::available('ZikulaCategoriesModule')) { 1545c1548 < $resultRows = $result->fetchAll(Doctrine::FETCH_NUM); --- > $resultRows = $result->fetchAll(Doctrine_Core::FETCH_NUM); 1595c1598 < $resultRows = $result->fetchAll(Doctrine::FETCH_ASSOC); --- > $resultRows = $result->fetchAll(Doctrine_Core::FETCH_ASSOC); 1945c1948 < if (!ModUtil::dbInfoLoad('Categories')) { --- > if (!ModUtil::dbInfoLoad('ZikulaCategoriesModule')) { 2382c2385 < $res = $res->fetchAll(Doctrine::FETCH_COLUMN); // RNG: Should this really be fetchAll() ?? --- > $res = $res->fetchAll(Doctrine_Core::FETCH_COLUMN); // RNG: Should this really be fetchAll() ?? 2782c2785 < $res = $res->fetchAll(Doctrine::FETCH_COLUMN); --- > $res = $res->fetchAll(Doctrine_Core::FETCH_COLUMN); 2904c2907 < if (($enableAllServices || (isset($tables["{$table}_db_extra_enable_categorization"]) && $tables["{$table}_db_extra_enable_categorization"])) && System::getVar('Z_CONFIG_USE_OBJECT_CATEGORIZATION') && strcmp($table, 'categories_') !== 0 && strcmp($table, 'objectdata_attributes') !== 0 && strcmp($table, 'objectdata_log') !== 0 && ModUtil::available('Categories')) { --- > if (($enableAllServices || (isset($tables["{$table}_db_extra_enable_categorization"]) && $tables["{$table}_db_extra_enable_categorization"])) && System::getVar('Z_CONFIG_USE_OBJECT_CATEGORIZATION') && strcmp($table, 'categories_') !== 0 && strcmp($table, 'objectdata_attributes') !== 0 && strcmp($table, 'objectdata_log') !== 0 && ModUtil::available('ZikulaCategoriesModule')) { 3914c3917 < return z_exit(__f('%1$s: unable to limit tablename [%2$s] because database prefix is too long for Oracle, please shorten it (recommended length is 4 chars)', array( --- > throw new \Exception(__f('%1$s: unable to limit tablename [%2$s] because database prefix is too long for Oracle, please shorten it (recommended length is 4 chars)', array( 4006c4009 < } --- > } \ No newline at end of file ``` espaan
…eader View plugin {moduleheader}
…header User theme name in moduleheader
See ef34c5b where this was also applied in 1.4.x to solve for instance Bootstrap Tabs disappearing when also using Prototype. For users using Bootstrap (themes) in 1.3.x this solves issues.
Updated noconflict for resolving prototype/jQuery/Bootstrap issues
…tect is also enabled, but client is not mobile
…tect is also enabled, but client is not mobile
…m_Printer_RSS [1.3] Load correct theme in case Mobile theme is enabled and Mobile detect is enabled
…m_Printer_RSS Changelog entry for #2583
To use the filter-function in the generated link
Include in the {ext} function filtering for CLIP
…UserUtil [1.3] Typo correction in UserUtil.php
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.