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.

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

QueryBuilder output columns incorrect #48521

Closed
4 of 8 tasks
tobiasKaminsky opened this issue Oct 2, 2024 · 1 comment · Fixed by #48522
Closed
4 of 8 tasks

QueryBuilder output columns incorrect #48521

tobiasKaminsky opened this issue Oct 2, 2024 · 1 comment · Fixed by #48522
Assignees
Labels

Comments

@tobiasKaminsky
Copy link
Member

⚠️ This issue respects the following points: ⚠️

Bug description

Use master

Steps to reproduce

  1. Use master
  2. try to update your SSE password
 "File": "/var/www/html/lib/private/DB/QueryBuilder/QueryBuilder.php",
    "Line": 217,
    "message": "More than 30 characters for an output column name are not allowed on Oracle.",
    "query": "SELECT `c`.*, `r`.`id` as `node_rel_id`, `r`.`node_id`, `r`.`node_type`, `r`.`permissions`, `p`.`id` as `page_id`, `p`.`page_type`, `pc`.`id` as `content_id`, `pc`.`order`, `n`.`display_mode` as `display_mode_default`, `s`.`id` as `share_id`, `s`.`receiver`, `s`.`receiver_type`, `n2`.`display_mode` FROM `*PREFIX*tables_contexts_context` `c` LEFT JOIN `*PREFIX*tables_shares` `s` ON (`c`.`id` = `s`.`node_id`) AND (`s`.`node_type` = :dcValue6) AND ((`s`.`sender` = :dcValue1) OR ((`s`.`receiver_type` = :dcValue2) AND (`s`.`receiver` = :dcValue3)) OR ((`s`.`receiver_type` = :dcValue4) AND (`s`.`receiver` IN (:dcValue5)))) LEFT JOIN `*PREFIX*tables_contexts_page` `p` ON `c`.`id` = `p`.`context_id` LEFT JOIN `*PREFIX*tables_contexts_rel_context_node` `r` ON `c`.`id` = `r`.`context_id` LEFT JOIN `*PREFIX*tables_contexts_navigation` `n2` ON (`s`.`id` = `n2`.`share_id`) AND (`n2`.`user_id` = :dcValue8) LEFT JOIN `*PREFIX*tables_contexts_navigation` `n` ON (`s`.`id` = `n`.`share_id`) AND (`n`.`user_id` = :dcValue9) LEFT JOIN `*PREFIX*tables_contexts_page_content` `pc` ON `p`.`id` = `pc`.`page_id` WHERE ((`owner_id` = :dcValue7) OR (`s`.`receiver` IS NOT NULL)) AND ((`pc`.`node_rel_id` = `r`.`id`) OR (`pc`.`node_rel_id` IS NULL)) ORDER BY `pc`.`order` ASC",
    "columns": [
      "display_mode as display_mode_default"
    ],

Expected behavior

No error.

Nextcloud Server version

master

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@tobiasKaminsky tobiasKaminsky added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Oct 2, 2024
@nickvergessen nickvergessen transferred this issue from nextcloud/server Oct 2, 2024
@nickvergessen nickvergessen changed the title [Bug]: encryption app: More than 30 characters for an output column name are not allowed More than 30 characters for an output column name are not allowed Oct 2, 2024
@provokateurin provokateurin self-assigned this Oct 2, 2024
@provokateurin provokateurin added 2. developing Work in progress and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Oct 2, 2024
@provokateurin
Copy link
Member

The column length is actually fine, just the error is wrong as it includes the whole ... as ... and not just the real column name which is only the last part.

@provokateurin provokateurin transferred this issue from nextcloud/tables Oct 2, 2024
@provokateurin provokateurin changed the title More than 30 characters for an output column name are not allowed QueryBuilder output columns incorrect Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ☑️ Done
Development

Successfully merging a pull request may close this issue.

3 participants