-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
SQL query contains duplicate order statement when using groups, 500 error on 2.3 / 2.4 #3418
Comments
If anyone can point me in the direction of what file might be incorrectly formatting the select statement, I can take a look and see if I can figure out. I've been digging through the files but haven't been able to figure it out yet. |
I'll take a look at this now, though I don't have an MS SQL Server to test against. Would you be able to pull down a branch and test for me? |
@Joe-K2018 can you test out #3432 please. |
Looks good! I can load the status page after adding an incident now in 2.4. I am still having this issue after adding components to groups and trying to view the status page, but I might have fixed it? I was playing around with this last night. I'm new to PHP programming but got XDebug set up and was going through it. I ended up removing the order by statement from two locations here: https://i.imgur.com/InwEeNV.png It worked after that, but I'm not sure if that is the right way to go about it. The names of the files appear to be random, so I'm thinking these must be auto-generated files. If that's the case, I'm worried about running into this issue again when I do a fresh install on a production server. What do you think? |
I think it'll likely be the same issue, just in different presenters. Bear with! |
Okay, pull down the branch again :) |
It's better! I can view the status page if I go to the base url, but if I go to /dashboard I am still receiving the error.
|
Can you provide me with the full log file please? Also, try running |
* New translations cachet.php (Portuguese) * New translations cachet.php (Polish) * New translations cachet.php (Romanian) * New translations cachet.php (Japanese) * New translations cachet.php (Korean) * New translations cachet.php (Norwegian) * New translations forms.php (Norwegian) * New translations cachet.php (Persian) * New translations cachet.php (Russian) * New translations cachet.php (Ukrainian) * New translations cachet.php (Vietnamese) * New translations cachet.php (Zulu) * New translations forms.php (Zulu) * New translations cachet.php (Estonian) * New translations cachet.php (Spanish) * New translations cachet.php (Swedish) * New translations cachet.php (Thai) * New translations cachet.php (Turkish) * New translations cachet.php (Chinese Traditional) * New translations cachet.php (Czech) * New translations cachet.php (Danish) * New translations cachet.php (Dutch) * New translations forms.php (Dutch) * New translations forms.php (Chinese Simplified) * New translations cachet.php (Arabic) * New translations cachet.php (Albanian) * New translations cachet.php (Catalan) * New translations cachet.php (Chinese Simplified) * New translations cachet.php (Hebrew) * New translations cachet.php (Hungarian) * New translations cachet.php (Indonesian) * New translations cachet.php (Italian) * New translations cachet.php (Greek) * New translations cachet.php (English) * New translations cachet.php (Finnish) * New translations cachet.php (French) * New translations cachet.php (German) * Fix configuring postgres leads to wrong database driver key * Add warning when setting up localhost with mysql driver * Explicitly set the port number when not using traditional ports * Add support for empty values in the persistEnv method * [Security] Bump url-parse from 1.4.1 to 1.4.4 Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.4.1 to 1.4.4. **This update includes security fixes.** - [Release notes](https://github.com/unshiftio/url-parse/releases) - [Commits](unshiftio/url-parse@1.4.1...1.4.4) Signed-off-by: dependabot[bot] <support@dependabot.com> * Use PHP Debugger(phpdbg) for PHPUnit to speed up CI * Improve database performance by removing duplicated queries * StyleCI * Eager load user on the incident page not to query twice * Eager load group on the dashboard/components page not to query twice * Improve database performance by removing duplicated queries * StyleCI * Resolve cachethq#3269 * Resolve cachethq#3288 If Binput::get('enabled') is not defined this will return NULL which resulted in a false. Updated tests * New translations cachet.php (Swedish) * New translations dashboard.php (Swedish) * Apply fixes from StyleCI * Update deps * Drop support for RSS & Atom feeds. * Update Blade files * Don't expose current mail password * Rebuilt composer.lock * Bump laravolt/avatar from 1.8.1 to 2.1.0 Bumps [laravolt/avatar](https://github.com/laravolt/avatar) from 1.8.1 to 2.1.0. - [Release notes](https://github.com/laravolt/avatar/releases) - [Changelog](https://github.com/laravolt/avatar/blob/master/CHANGELOG.md) - [Commits](laravolt/avatar@1.8.1...2.1.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel-mix-purgecss from 2.2.0 to 3.0.0 Bumps [laravel-mix-purgecss](https://github.com/spatie/laravel-mix-purgecss) from 2.2.0 to 3.0.0. - [Release notes](https://github.com/spatie/laravel-mix-purgecss/releases) - [Changelog](https://github.com/spatie/laravel-mix-purgecss/blob/master/CHANGELOG.md) - [Commits](spatie/laravel-mix-purgecss@2.2.0...3.0.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Upgraded JS deps and rebuilt assets * GitHub now supports transferring an issue * Remove unused $config property * Apply fixes from StyleCI * Upgraded to Laravel 5.7 * Deal with settings read/write errors properly * Apply fixes from StyleCI * Commit new vendor files * Apply fixes from StyleCI * Fixed up vendor views * Updated doctrine and emoji packages * Set the mail username optional in setup During the setup process email informations were asked: - Driver - Host - Username - Password In some situations the username is not useful because the Cachet's host may be configured to forward email to a server. The problem is the username was required, so we had to set a username and then update the .env file to remove it. To fix this problem, the mail username has been set to optional in the setup. So if someone needs a username it still can use this field, and otherwise people can let it empty. See: cachethq#3244 * Let the mail_username required except for sendmail During the setup the "mail_username" was required and it was then undone, so using the sendmail driver we can let the username empty. It would be bad to let the username optional for every drivers, because in some configurations, like SMTP, the username is required for the SMTP server so if the user let it empty its mail configuration will be bad. The mail_username is now optional only if the mail driver is sendmail. See: cachethq#3244 * Apply fixes from StyleCI * Upgraded dependencies following XSS vulnerability discovery * Use safer markdown defaults * [Security] Bump twig/twig from 1.36.0 to 2.6.0 Bumps [twig/twig](https://github.com/twigphp/Twig) from 1.36.0 to 2.6.0. **This update includes security fixes.** - [Release notes](https://github.com/twigphp/Twig/releases) - [Changelog](https://github.com/twigphp/Twig/blob/2.x/CHANGELOG) - [Commits](twigphp/Twig@v1.36.0...v2.6.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * New translations forms.php (Dutch) * New translations cachet.php (German) * New translations pagination.php (German) * Apply fixes from StyleCI * Bump doctrine/dbal from 2.9.1 to 2.9.2 Bumps [doctrine/dbal](https://github.com/doctrine/dbal) from 2.9.1 to 2.9.2. - [Release notes](https://github.com/doctrine/dbal/releases) - [Commits](doctrine/dbal@v2.9.1...v2.9.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel-mix and laravel-mix-purgecss Bumps [laravel-mix](https://github.com/JeffreyWay/laravel-mix) and [laravel-mix-purgecss](https://github.com/spatie/laravel-mix-purgecss). These dependencies needed to be updated together. Updates `laravel-mix` from 2.1.14 to 4.0.12 - [Release notes](https://github.com/JeffreyWay/laravel-mix/releases) - [Commits](laravel-mix/laravel-mix@v2.1.14...v4.0.12) Updates `laravel-mix-purgecss` from 3.0.0 to 4.0.0 - [Release notes](https://github.com/spatie/laravel-mix-purgecss/releases) - [Changelog](https://github.com/spatie/laravel-mix-purgecss/blob/master/CHANGELOG.md) - [Commits](spatie/laravel-mix-purgecss@3.0.0...4.0.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump LICENSE year * New translations validation.php (Dutch) * Link to Exascale sponsor * Extra dependencies were required * Revert "Bump laravel-mix and laravel-mix-purgecss" * Remove Gulp and update requirements lists * Use Twig namespaces * Move overview up * Add documentation * Improve database performance by removing duplicated queries * Typo in phpdoc * Revert "Typo in phpdoc" This reverts commit af94758. * Typo in phpdoc * Bump symfony/dom-crawler from 4.2.1 to 4.2.2 Bumps [symfony/dom-crawler](https://github.com/symfony/dom-crawler) from 4.2.1 to 4.2.2. - [Release notes](https://github.com/symfony/dom-crawler/releases) - [Changelog](https://github.com/symfony/dom-crawler/blob/master/CHANGELOG.md) - [Commits](symfony/dom-crawler@v4.2.1...v4.2.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.82.3 to 3.82.6 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.82.3 to 3.82.6. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.82.3...3.82.6) Signed-off-by: dependabot[bot] <support@dependabot.com> * New translations forms.php (Dutch) * New translations dashboard.php (Dutch) * New translations validation.php (Dutch) * New translations cachet.php (Dutch) * Bump fideloper/proxy from 4.0.0 to 4.1.0 Bumps [fideloper/proxy](https://github.com/fideloper/TrustedProxy) from 4.0.0 to 4.1.0. - [Release notes](https://github.com/fideloper/TrustedProxy/releases) - [Commits](fideloper/TrustedProxy@4.0.0...4.1.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * [Security] Bump twig/twig from 2.6.0 to 2.6.2 Bumps [twig/twig](https://github.com/twigphp/Twig) from 2.6.0 to 2.6.2. **This update includes security fixes.** - [Release notes](https://github.com/twigphp/Twig/releases) - [Changelog](https://github.com/twigphp/Twig/blob/2.x/CHANGELOG) - [Commits](twigphp/Twig@v2.6.0...v2.6.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.82.6 to 3.86.2 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.82.6 to 3.86.2. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.82.6...3.86.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel/framework from 5.7.19 to 5.7.21 Bumps [laravel/framework](https://github.com/laravel/framework) from 5.7.19 to 5.7.21. - [Release notes](https://github.com/laravel/framework/releases) - [Changelog](https://github.com/laravel/framework/blob/5.7/CHANGELOG-5.7.md) - [Commits](laravel/framework@v5.7.19...v5.7.21) Signed-off-by: dependabot[bot] <support@dependabot.com> * New translations cachet.php (German) * New translations dashboard.php (German) * New translations forms.php (German) * New translations notifications.php (German) * Remove duplicate order statement. Fixes cachethq#3418 * Remove string about HTTP(s). Fixes cachethq#3421 * Fixes CORS headers. Closes cachethq#3413 * Apply fixes from StyleCI * Update deps * Compile assets * Fix importing of node_modules CSS * Compile assets * Fully remove the Strengthify integration * Compile assets * New translations cachet.php (Mongolian) * New translations dashboard.php (Mongolian) * New translations forms.php (Mongolian) * New translations pagination.php (Mongolian) * New translations setup.php (Mongolian) * New translations validation.php (Mongolian) * New translations notifications.php (Mongolian) * New translations cachet.php (Slovenian) * New translations dashboard.php (Slovenian) * New translations forms.php (Slovenian) * New translations pagination.php (Slovenian) * New translations setup.php (Slovenian) * New translations validation.php (Slovenian) * New translations notifications.php (Slovenian) * Fix other duplicate order by statements * Apply fixes from StyleCI * Fix HTML encoding for Google Fonts * Update blog feed Fixes cachethq#3441 * Bump sortablejs from 1.7.0 to 1.8.1 Bumps [sortablejs](https://github.com/SortableJS/Sortable) from 1.7.0 to 1.8.1. - [Release notes](https://github.com/SortableJS/Sortable/releases) - [Commits](SortableJS/Sortable@1.7.0...1.8.1) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump vue and vue-template-compiler Bumps [vue](https://github.com/vuejs/vue) and [vue-template-compiler](https://github.com/vuejs/vue). These dependencies needed to be updated together. Updates `vue` from 2.5.21 to 2.5.22 - [Release notes](https://github.com/vuejs/vue/releases) - [Commits](vuejs/vue@v2.5.21...v2.5.22) Updates `vue-template-compiler` from 2.5.21 to 2.5.22 - [Release notes](https://github.com/vuejs/vue/releases) - [Commits](vuejs/vue@v2.5.21...v2.5.22) Signed-off-by: dependabot[bot] <support@dependabot.com> * Do not use arr_* or str_* helpers * Add missing Arr import * Fix setup username validation * Add user on cachet install command * Get a fresh configuration * Use class notation * Apply fixes from StyleCI * Don't prompt for tag files [Fix cachethq#3453] * Substitute :name by incident name on slack title Slack title for created incident has ':name', which is being substituted by incident name. * We no longer need the dev symfony requirements * Bump phpunit/phpunit from 7.5.3 to 7.5.4 Bumps [phpunit/phpunit](https://github.com/sebastianbergmann/phpunit) from 7.5.3 to 7.5.4. - [Release notes](https://github.com/sebastianbergmann/phpunit/releases) - [Changelog](https://github.com/sebastianbergmann/phpunit/blob/master/ChangeLog-7.5.md) - [Commits](sebastianbergmann/phpunit@7.5.3...7.5.4) Signed-off-by: dependabot[bot] <support@dependabot.com> * Change to appropriate tags to hide and select/unselect components in group * Drop dependency on alt-three throttle (cachethq#3469) I've just pulled in the subset of the features we were actually using. * Bump laravel/framework from 5.7.25 to 5.7.26 Bumps [laravel/framework](https://github.com/laravel/framework) from 5.7.25 to 5.7.26. - [Release notes](https://github.com/laravel/framework/releases) - [Changelog](https://github.com/laravel/framework/blob/5.7/CHANGELOG-5.7.md) - [Commits](laravel/framework@v5.7.25...v5.7.26) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump graham-campbell/binput from 6.0.0 to 6.1.0 Bumps [graham-campbell/binput](https://github.com/GrahamCampbell/Laravel-Binput) from 6.0.0 to 6.1.0. - [Release notes](https://github.com/GrahamCampbell/Laravel-Binput/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-Binput/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-Binput@v6.0.0...v6.1.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump graham-campbell/markdown from 10.2.0 to 10.3.0 Bumps [graham-campbell/markdown](https://github.com/GrahamCampbell/Laravel-Markdown) from 10.2.0 to 10.3.0. - [Release notes](https://github.com/GrahamCampbell/Laravel-Markdown/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-Markdown/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-Markdown@v10.2.0...v10.3.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump graham-campbell/testbench-core from 3.0.0 to 3.0.2 Bumps [graham-campbell/testbench-core](https://github.com/GrahamCampbell/Laravel-TestBench-Core) from 3.0.0 to 3.0.2. - [Release notes](https://github.com/GrahamCampbell/Laravel-TestBench-Core/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-TestBench-Core/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-TestBench-Core@v3.0.0...v3.0.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Re-enable middleware. Fixes cachethq#3472 * Move meta relation into trait * Remove badly added components class * Improve count call * More UX and UI fixes * Compile assets * [Security] Bump bootstrap-sass from 3.4.0 to 3.4.1 Bumps [bootstrap-sass](https://github.com/twbs/bootstrap-sass) from 3.4.0 to 3.4.1. **This update includes security fixes.** - [Release notes](https://github.com/twbs/bootstrap-sass/releases) - [Changelog](https://github.com/twbs/bootstrap-sass/blob/master/CHANGELOG.md) - [Commits](twbs/bootstrap-sass@v3.4.0...v3.4.1) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump mockery/mockery from 1.2.1 to 1.2.2 Bumps [mockery/mockery](https://github.com/mockery/mockery) from 1.2.1 to 1.2.2. - [Release notes](https://github.com/mockery/mockery/releases) - [Changelog](https://github.com/mockery/mockery/blob/master/CHANGELOG.md) - [Commits](mockery/mockery@1.2.1...1.2.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravolt/avatar from 2.1.0 to 2.2.0 Bumps [laravolt/avatar](https://github.com/laravolt/avatar) from 2.1.0 to 2.2.0. - [Release notes](https://github.com/laravolt/avatar/releases) - [Changelog](https://github.com/laravolt/avatar/blob/master/CHANGELOG.md) - [Commits](laravolt/avatar@2.1.0...2.2.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump phpunit/phpunit from 7.5.4 to 7.5.6 Bumps [phpunit/phpunit](https://github.com/sebastianbergmann/phpunit) from 7.5.4 to 7.5.6. - [Release notes](https://github.com/sebastianbergmann/phpunit/releases) - [Changelog](https://github.com/sebastianbergmann/phpunit/blob/master/ChangeLog-7.5.md) - [Commits](sebastianbergmann/phpunit@7.5.4...7.5.6) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.87.8 to 3.87.17 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.87.8 to 3.87.17. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.87.8...3.87.17) Signed-off-by: dependabot[bot] <support@dependabot.com> * Correctly retrieve the value * Remove duplicate CSS properties Remove unused parameter * Bump barryvdh/laravel-cors from 0.11.2 to 0.11.3 Bumps [barryvdh/laravel-cors](https://github.com/barryvdh/laravel-cors) from 0.11.2 to 0.11.3. - [Release notes](https://github.com/barryvdh/laravel-cors/releases) - [Changelog](https://github.com/barryvdh/laravel-cors/blob/master/changelog.md) - [Commits](fruitcake/laravel-cors@v0.11.2...v0.11.3) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump barryvdh/laravel-debugbar from 3.2.2 to 3.2.3 Bumps [barryvdh/laravel-debugbar](https://github.com/barryvdh/laravel-debugbar) from 3.2.2 to 3.2.3. - [Release notes](https://github.com/barryvdh/laravel-debugbar/releases) - [Changelog](https://github.com/barryvdh/laravel-debugbar/blob/master/changelog.md) - [Commits](barryvdh/laravel-debugbar@v3.2.2...v3.2.3) Signed-off-by: dependabot[bot] <support@dependabot.com> * Make pgsql schema configurable via environment variable * Bump graham-campbell/exceptions from 11.1.0 to 11.2.0 Bumps [graham-campbell/exceptions](https://github.com/GrahamCampbell/Laravel-Exceptions) from 11.1.0 to 11.2.0. - [Release notes](https://github.com/GrahamCampbell/Laravel-Exceptions/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-Exceptions/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-Exceptions@v11.1.0...v11.2.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.87.17 to 3.89.1 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.87.17 to 3.89.1. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.87.17...3.89.1) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel/framework from 5.7.26 to 5.7.28 Bumps [laravel/framework](https://github.com/laravel/framework) from 5.7.26 to 5.7.28. - [Release notes](https://github.com/laravel/framework/releases) - [Changelog](https://github.com/laravel/framework/blob/5.8/CHANGELOG-5.7.md) - [Commits](laravel/framework@v5.7.26...v5.7.28) Signed-off-by: dependabot[bot] <support@dependabot.com>
When I add a component to a group and try to go to the status page it crashes with a 500 error.
The same error is thrown in 2.3 as well as 2.4. In 2.3 and 2.4 it occurs after adding a component to a group. In 2.4 it also occurs after adding an incident.
I have a feeling this is related to me running MS SQL server which I know is not officially supported, but since that is what my company uses I would really like to get this working and would appreciate assistance.
It looks like the SQL query that is generated has a duplicate order statement which SQL server doesn't like.
Stacktrace from 2.3 after adding component to group:
Stacktrace from 2.4 after adding incident:
The text was updated successfully, but these errors were encountered: