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

Link type column data reset after row update #1036

Open
Jednadvacet opened this issue Apr 30, 2024 · 5 comments
Open

Link type column data reset after row update #1036

Jednadvacet opened this issue Apr 30, 2024 · 5 comments
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working

Comments

@Jednadvacet
Copy link

Steps to reproduce

  1. Fill data column of existing row. The column type should be link. My row has three link fields.
  2. Save row.
  3. Row saved. Link fields are cleared.

Expected behavior

Row update saves all fields.

Actual behavior

Link fields are cleared.

Tables app version

0.7.1.

Browser

Edge

Client operating system

No response

Operating system

Ubuntu 20.04

Web server

Nginx

PHP engine version

PHP 8.1

Database

PostgreSQL

Additional info

No response

@Jednadvacet Jednadvacet added 0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working labels Apr 30, 2024
@Jednadvacet
Copy link
Author

Jednadvacet commented May 9, 2024

Seems that the issue is connected with charset. Typical Czech language characters ščřžýáíěé in a link title call this errror:

Value {"title":"Ing. Tom\u00e1\u0161 \u0160enky\u0159\u00edk","value":"https:\/\/cloud.lbtechnet.cz\/apps\/contacts\/direct\/contact\/b08ce89e-2f35-48e5-b499-484a23185658~contacts","providerId":"url"} cannot be parsed as the column 52 does not allow the provider: url

New row is saved properly, update resets.

0.7.2. too.

@Jednadvacet
Copy link
Author

Quick fix: Link type fields should have at least 2 subtypes (eg. Files + Url).
Saving rows doesn't clear other link fields anymore.

@juliushaertl
Copy link
Member

Thanks, we should of course fix that to not empty the value and provide a proper hint on why that fails.

@juliushaertl
Copy link
Member

Might be worth to reconsider if the subtypes should actually limit what you can insert into an URL field or if those should just be used for search suggestions.

@lars-becker
Copy link

Quick fix: Link type fields should have at least 2 subtypes (eg. Files + Url). Saving rows doesn't clear other link fields anymore.

Thanks that solved the problem with repeated data loss for us. This problem should be fixed ASAP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants