Project

General

Profile

Actions

Bug #25232

closed

CV History: Incorrect API call due to parsing error (page=NaN)

Added by Mirek Długosz over 5 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Category:
Content Views
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Content view History tab sends following API request:
/katello/api/v2/content_views/4/history?organization_id=1&page=NaN&paged=true&per_page=20&search=&sort_by=created_at&sort_order=DESC

This is currently shadowed by issue #25231, but once it is fixed, we will see UI complaining about invalid value for Integer(): "NaN".

Katello git commit id: 77a4d76db2d1524f152281a78ef8d17f4fdfa646
Foreman git commit id: 9ccfbde5c417ed466b21c997d01d4608cc4e8c57

Steps to reproduce:
1. Create new content view
2. Open this content view and navigate to History tab

Expected results:
There is no error notification; history data is displayed in table (if there is any history to display).


Related issues 1 (0 open1 closed)

Related to Katello - Bug #25231: "the field 'created_at' in the order statement is not valid field for search" on content view history tabClosedChristine FouantActions
Actions #1

Updated by Mirek Długosz over 5 years ago

  • Related to Bug #25231: "the field 'created_at' in the order statement is not valid field for search" on content view history tab added
Actions #2

Updated by Tomer Brisker over 5 years ago

  • Project changed from Foreman to Katello
  • Category deleted (Web Interface)
Actions #3

Updated by John Mitsch over 5 years ago

  • Category set to Content Views
  • Target version set to Katello 3.11.0
  • Triaged changed from No to Yes
Actions #4

Updated by Jonathon Turel over 5 years ago

  • Assignee set to Jonathon Turel
Actions #5

Updated by Jonathon Turel over 5 years ago

Hey Mirek, I think there was a corresponding Foreman bug which has been addressed and fixed this issue. Take another look and we can close this one if it's taken care of.

Actions #6

Updated by Mirek Długosz over 5 years ago

  • Status changed from New to Resolved

Jonathon,

I can no longer reproduce it as of:
Foreman git id 15ac394288f0719f4d738301370836ed0a7007f5
Katello git id 6dd89e862945a20e56d972a4b030868ac744b66b

You are right, this must have been addressed by another commit already.

Downstream Satellite is not affected either.

Actions #7

Updated by Mc Glynn over 3 years ago

Incorrect API calls mean that your request will be rejected. If you had asked for custom writings then your site will display pain text. Your site will look like any normal website and still be operational.

Actions

Also available in: Atom PDF