Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Switching a field type from Checkbox to Lightswitch causes MySQL error about data truncation #1

Closed
angrybrad opened this issue Jan 27, 2017 · 1 comment
Closed

Switching a field type from Checkbox to Lightswitch causes MySQL error about data truncation #1

angrybrad opened this issue Jan 27, 2017 · 1 comment
Labels
bug mysql severity:minor Cosmetic issues or bugs with simple workarounds

Comments

@angrybrad
Copy link
Member

angrybrad commented Jan 27, 2017 •
edited
Loading

Environment:

  • Craft version: 3.x
  • PHP version: 7.x
  • Database driver & version: MySQL - Need to verify behavior in PostgreSQL
  • Plugins & versions:

Description:

Switching a field type from Checkbox to Lightswitch causes MySQL error about data truncation.

Steps to reproduce:

Change a field with data in it from a Checkbox to a Lightswitch.
@angrybrad
Copy link
Member Author

Maybe come up with a way for field types to be notified when their type changes. Also maybe a the concept of "convertible/compatible" data types for field types when changing.

@angrybrad angrybrad added bug mysql severity:normal Bugs that affect typical usage labels Jan 27, 2017
This was referenced Jan 27, 2017
Closed
Closed
@johanneslamers johanneslamers mentioned this issue Feb 1, 2017
Closed
@my2ter my2ter mentioned this issue Feb 1, 2017
Closed
@brandonkelly brandonkelly added severity:minor Cosmetic issues or bugs with simple workarounds and removed severity:normal Bugs that affect typical usage labels Feb 3, 2017
@johnfmorton johnfmorton mentioned this issue Feb 7, 2017
Closed
@narration-sd narration-sd mentioned this issue Feb 7, 2017
Closed
@alambc alambc mentioned this issue Feb 22, 2017
Closed
@narration-sd narration-sd mentioned this issue Mar 30, 2017
Closed
@damienbuckley damienbuckley mentioned this issue May 12, 2017
Closed
leevigraham referenced this issue in leevigraham/cms Jun 1, 2017
@leevigraham
Use entry model to get ID rather than variable
2ad1e23 
This fixes an issue where changing the entryType of an existing entry triggers a 500 error:

PHP Notice 'yii\base\ErrorException' with message 'Undefined index: entryId' 

in /Volumes/Sites/Personal/craft3/vendor/craftcms/cms/src/controllers/EntriesController.php:773

Stack trace:
#0 … vendor/yiisoft/yii2/base/InlineAction.php(57): craft\controllers\EntriesController->actionSwitchEntryType()
#1 … vendor/yiisoft/yii2/base/InlineAction.php(57): ::call_user_func_array:{/Volumes/Sites/Personal/craft3/vendor/yiisoft/yii2/base/InlineAction.php:57}('???', '???')
#2 … vendor/yiisoft/yii2/base/Controller.php(156): yii\base\InlineAction->runWithParams('???')
craftcms#3 … vendor/yiisoft/yii2/base/Module.php(523): yii\base\Controller->runAction('???', '???')
craftcms#4 … vendor/craftcms/cms/src/web/Application.php(246): yii\base\Module->runAction('???', '???')
craftcms#5 … vendor/craftcms/cms/src/web/Application.php(385): craft\web\Application->runAction('???', '???')
craftcms#6 … vendor/craftcms/cms/src/web/Application.php(206): craft\web\Application->_processActionRequest('???')
craftcms#7 … vendor/yiisoft/yii2/base/Application.php(380): craft\web\Application->handleRequest('???')
craftcms#8 … web/index.php(33): yii\base\Application->run()
craftcms#9 {main}
brandonkelly pushed a commit that referenced this issue Jun 1, 2017
@leevigraham @brandonkelly
Use entry model to get ID rather than variable
53c6d6c 
This fixes an issue where changing the entryType of an existing entry triggers a 500 error:

PHP Notice 'yii\base\ErrorException' with message 'Undefined index: entryId' 

in /Volumes/Sites/Personal/craft3/vendor/craftcms/cms/src/controllers/EntriesController.php:773

Stack trace:
#0 … vendor/yiisoft/yii2/base/InlineAction.php(57): craft\controllers\EntriesController->actionSwitchEntryType()
#1 … vendor/yiisoft/yii2/base/InlineAction.php(57): ::call_user_func_array:{/Volumes/Sites/Personal/craft3/vendor/yiisoft/yii2/base/InlineAction.php:57}('???', '???')
#2 … vendor/yiisoft/yii2/base/Controller.php(156): yii\base\InlineAction->runWithParams('???')
#3 … vendor/yiisoft/yii2/base/Module.php(523): yii\base\Controller->runAction('???', '???')
#4 … vendor/craftcms/cms/src/web/Application.php(246): yii\base\Module->runAction('???', '???')
#5 … vendor/craftcms/cms/src/web/Application.php(385): craft\web\Application->runAction('???', '???')
#6 … vendor/craftcms/cms/src/web/Application.php(206): craft\web\Application->_processActionRequest('???')
#7 … vendor/yiisoft/yii2/base/Application.php(380): craft\web\Application->handleRequest('???')
#8 … web/index.php(33): yii\base\Application->run()
#9 {main}
@andrelopez andrelopez mentioned this issue Jul 7, 2017
Closed
@andrelopez andrelopez mentioned this issue Aug 30, 2017
Closed
brandonkelly pushed a commit that referenced this issue Sep 11, 2017
@markhuot
Merge pull request #1 from craftcms/develop
5a849a3 
update
angrybrad pushed a commit that referenced this issue Oct 4, 2017
@ebeauchamps
Merge pull request #1 from craftcms/develop
ddbf5cc 
Merging from upstream
@retop retop mentioned this issue Nov 24, 2017
Closed
@E2design E2design mentioned this issue Dec 8, 2017
Open
@aashish-pixere aashish-pixere mentioned this issue Dec 28, 2017
Closed
@ddbr ddbr mentioned this issue Jan 25, 2018
Closed
@jonlivingston jonlivingston mentioned this issue Jan 26, 2018
Closed
@steveooo9 steveooo9 mentioned this issue Jan 30, 2018
Closed
@Rainbowlemon Rainbowlemon mentioned this issue Feb 14, 2018
Closed
@leigeber leigeber mentioned this issue Feb 21, 2018
Closed
@iwfweb iwfweb mentioned this issue Feb 16, 2023
Closed
@DenyEs DenyEs mentioned this issue Apr 28, 2023
Closed
@matthaltuch matthaltuch mentioned this issue May 30, 2023
Closed
@joepagan joepagan mentioned this issue Jun 8, 2023
Closed
@leowebguy leowebguy mentioned this issue Aug 25, 2023
Closed
@zzseba78 zzseba78 mentioned this issue Aug 25, 2023
Closed
@moldedjelly moldedjelly mentioned this issue Sep 19, 2023
Closed
@bowen31337 bowen31337 mentioned this issue Oct 5, 2023
Open
@rw-ninety rw-ninety mentioned this issue Oct 21, 2023
Closed
@robinbeatty robinbeatty mentioned this issue Oct 31, 2023
Closed
@DTesch-Reem DTesch-Reem mentioned this issue Jan 9, 2024
Closed
@proimage proimage mentioned this issue Jan 24, 2024
Closed
@MatthiasBrodelet MatthiasBrodelet mentioned this issue Feb 9, 2024
Closed
@hhentschel hhentschel mentioned this issue Feb 15, 2024
Closed
timkelty added a commit that referenced this issue Feb 23, 2024
@timkelty
Avoid file_exists deprecation warning
4f15949 
file_exists(): Passing null to parameter #1 ($filename) of type string is deprecated
@timkelty timkelty mentioned this issue Feb 23, 2024
Merged
@samdi samdi mentioned this issue Apr 10, 2024
Closed
@LinneaHarts LinneaHarts mentioned this issue Apr 26, 2024
Closed
@panlatent panlatent mentioned this issue Jun 3, 2024
Closed
@loqus loqus mentioned this issue Jun 14, 2024
Closed
@kevincote kevincote mentioned this issue Jul 24, 2024
Closed
@chadcrowell chadcrowell mentioned this issue Aug 6, 2024
Closed
@funkysoul funkysoul mentioned this issue Aug 6, 2024
Closed
@pnpaes pnpaes mentioned this issue Aug 23, 2024
Closed
@hatyi hatyi mentioned this issue Sep 12, 2024
Closed
@stephenmatthews stephenmatthews mentioned this issue Oct 24, 2024
Open
@joshmcoffman joshmcoffman mentioned this issue Oct 31, 2024
Open
@fancy-pug fancy-pug mentioned this issue Nov 13, 2024
Closed
@tibbis tibbis mentioned this issue Nov 13, 2024
Closed
@peter0503 peter0503 mentioned this issue Dec 12, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug mysql severity:minor Cosmetic issues or bugs with simple workarounds
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@brandonkelly @angrybrad