Bug 1831956 - RESTAPI javadoc contains malformed link around time zone representation
Summary: RESTAPI javadoc contains malformed link around time zone representation
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 4.3.9
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ovirt-4.4.2
: ---
Assignee: Ori Liel
QA Contact: Lucie Leistnerova
URL:
Whiteboard:
Depends On: 1831949 1831954 1833327
Blocks: 1831952
TreeView+ depends on / blocked
 
Reported: 2020-05-06 02:20 UTC by Rolfe Dlugy-Hegwer
Modified: 2023-09-15 01:29 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of: 1831954
Environment:
Last Closed: 2020-09-23 16:11:10 UTC
oVirt Team: Infra
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-36329 0 None None None 2022-07-09 12:57:40 UTC
Red Hat Product Errata RHSA-2020:3807 0 None None None 2020-09-23 16:12:06 UTC

Description Rolfe Dlugy-Hegwer 2020-05-06 02:20:58 UTC
+++ This bug was initially created as a clone of Bug #1831954 +++

Description of problem:

Javadoc contains a malformed link:

```
=== TimeZone [small]#struct#

Time zone representation.

.Attributes summary
[cols="20,20,60"]
|===
|Name |Type |Summary

...
|`utc_offset`
|<<types-string,String>>
|Offset from https://en. <<<<<<<<<<<<<<<<<<HERE
is installed in the virtual machine.

```



Version-Release number of selected component (if applicable):
4.3.9

How reproducible:
100%

Steps to Reproduce:
1.Open https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.3/html-single/rest_api_guide/
2. Ctrl+f search for "Offset from https://en."


Actual results:

Current URL produces a 404 error

Expected results:

URL should work.

Additional info:

Comment 1 Martin Perina 2020-05-07 07:55:37 UTC

*** This bug has been marked as a duplicate of bug 1831952 ***

Comment 2 Martin Perina 2020-05-07 12:32:25 UTC
OK, reopening, this has nothing to do with ovirt-engine-api-explorer, the javadoc is generated from ovirt-engine-api-model

Comment 3 Ori Liel 2020-05-21 07:44:40 UTC
Following the link provided in the bug description, it actually looks lie the reference to UTC is fine, not broken

Can you please check again? Perhaps this pertains to an earlier version?

Comment 4 Rolfe Dlugy-Hegwer 2020-05-22 10:57:09 UTC
Is the link formatted correctly? In other words, is it correctly enclosed in tags?
See https://stackoverflow.com/questions/1082050/linking-to-an-external-url-in-javadoc

Comment 5 Ori Liel 2020-05-25 06:59:33 UTC
The formatting is consistent with that of other links in ovirt-engine-api-model project. See the contents of the javadoc comment here:

     https://github.com/oVirt/ovirt-engine-api-model/blob/2a844bdf96a4c225e16e4963794f588d1a61fc50/src/main/java/types/TimeZone.java#L41

Also, unless I'm missing something, it works in the document that you have mentioned:

   https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.3/html-single/rest_api_guide/

Comment 6 Rolfe Dlugy-Hegwer 2020-05-26 13:49:51 UTC
Hi Ori. 

I put a temporary fix in the doc.

I should have also added a note in the bug description saying: "I am putting a temporary fix in the doc that will be erased next time we regenerate the API doc from the code. You must fix this in the source code.
Don't be fooled if you look at the current doc and it seems to be fixed."

To see the unfixed version, see: https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.2/html-single/rest_api_guide/index#types-time_zone

This SE topic explains how to format URLs in JavaDoc: https://stackoverflow.com/questions/1082050/linking-to-an-external-url-in-javadoc

It looks like we should enclose every URL in our Javadoc comments in an <a> element.

For example:
`<a href="https://en.wikipedia.org/wiki/Coordinated_Universal_Time">UTC</a>`

The current practice of using naked URLs, shown here, fails.
`Offset from https://en.wikipedia.org/wiki/Coordinated_Universal_Time[UTC].`

Would you:
1) Search for and fix naked URLs throughout the JavaDoc accordingly?
2) Add a note to any guides or standards that recommends enclosing URLs in <a> tags?

Comment 14 errata-xmlrpc 2020-09-23 16:11:10 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: Red Hat Virtualization security, bug fix, and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:3807

Comment 16 Red Hat Bugzilla 2023-09-15 01:29:30 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 365 days


Note You need to log in before you can comment on or make changes to this bug.