Skip to content

FIREFLY-166: fixed condition when Link Substitution fails, table fail to load. #840

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

Merged
merged 1 commit into from
Aug 8, 2019

Conversation

loitly
Copy link
Contributor

@loitly loitly commented Jul 19, 2019

https://jira.ipac.caltech.edu/browse/FIREFLY-166

When VOTable Link contains bad column reference, table should not fail to load.
This fix applies to both href and value substitution. When the substitution fail to resolve, it will use what is given.

To test:
https://irsawebdev9.ipac.caltech.edu/FIREFLY-166_link_substitution_error_handling/firefly/

  • upload the attached file in the ticket. In the 4th column, the 5th links should say
    info-${non-existing-column}-NGC6543, because non-existing-column is not a valid column in this table.

@loitly loitly requested a review from wmiipac July 19, 2019 22:25
@loitly loitly self-assigned this Jul 19, 2019
Copy link
Contributor

@wmiipac wmiipac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested, looks good. Thanks!

@ejoliet
Copy link
Contributor

ejoliet commented Jul 19, 2019

(ok, but in my local safari, firefox and chrome, i've tried to upload the table but it keep spinning...) - aprat from that, i also realized that the result table in the viewer shows extra column ID that looks suspiciously the continuation of the table_name... but is not part of the original VOtable.

@loitly
Copy link
Contributor Author

loitly commented Aug 5, 2019

(ok, but in my local safari, firefox and chrome, i've tried to upload the table but it keep spinning...) - aprat from that, i also realized that the result table in the viewer shows extra column ID that looks suspiciously the continuation of the table_name... but is not part of the original VOtable.

@ejoliet this sound like an issue not related to this PR. Can I go ahead and merge this PR and you create a new ticket for the issues you found?

@ejoliet
Copy link
Contributor

ejoliet commented Aug 5, 2019

Ok, np, i'll create a new ticket. Thanks.

@loitly loitly merged commit 8c13b86 into dev Aug 8, 2019
@loitly loitly deleted the FIREFLY-166_link_substitution_error_handling branch August 8, 2019 18:08
@robyww robyww added bug Table Changes to table or table model labels Oct 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Table Changes to table or table model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants