Update provenance links in browser

server/routes/browser/api.py

@@ -12,29 +12,34 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 """Knowledge Graph related handlers."""
-
-import json
-
 import flask
-from flask import request
-from flask import Response
+from flask import jsonify
 
 from server.lib import fetch
 from server.lib.cache import cache
 from server.routes import TIMEOUT
-import server.services.datacommons as dc
 
 bp = flask.Blueprint('api_browser', __name__, url_prefix='/api/browser')
 
 
 @bp.route('/provenance')
 @cache.cached(timeout=TIMEOUT, query_string=True)
 def provenance():
-  """Returns all the provenance information."""
+  """Return provenance name for all available data sources."""
   prov_resp = fetch.property_values(['Provenance'], 'typeOf', False)
-  url_resp = fetch.property_values(prov_resp['Provenance'], "url", True)
+  prov_dcids = prov_resp.get('Provenance', [])
+
+  if not prov_dcids:
+    return jsonify({})
+
+  properties_to_fetch = ['name']
+  prop_resp = fetch.multiple_property_values(prov_dcids, properties_to_fetch,
+                                             True)
+
   result = {}
-  for dcid, urls in url_resp.items():
-    if len(urls) > 0:
-      result[dcid] = urls[0]
-  return result
+  for dcid, props in prop_resp.items():
+    names = props.get('name', [])
+
+    result[dcid] = {'name': names[0] if names else dcid}
+
+  return jsonify(result)

static/js/browser/app.tsx

@@ -49,7 +49,7 @@ interface BrowserPagePropType {
 }
 
 interface BrowserPageStateType {
-  provDomain: { [key: string]: URL };
+  provenanceNames: { [key: string]: string };
   dataFetched: boolean;
 }
 
@@ -101,7 +101,7 @@ export class BrowserPage extends React.Component<
     super(props);
     this.state = {
       dataFetched: false,
-      provDomain: {},
+      provenanceNames: {},
     };
   }
 
@@ -163,7 +163,7 @@ export class BrowserPage extends React.Component<
             <h3>{outArcHeader}</h3>
             <OutArcSection
               dcid={arcDcid}
-              provDomain={this.state.provDomain}
+              provenanceNames={this.state.provenanceNames}
               nodeTypes={this.props.nodeTypes}
               showAllProperties={showAllProperties}
             />
@@ -186,7 +186,7 @@ export class BrowserPage extends React.Component<
             <InArcSection
               nodeName={this.props.nodeName}
               dcid={this.props.dcid}
-              provDomain={this.state.provDomain}
+              provenanceNames={this.state.provenanceNames}
             />
           )}
           {this.props.pageDisplayType === PageDisplayType.PLACE_STAT_VAR && (
@@ -222,18 +222,18 @@ export class BrowserPage extends React.Component<
       .get("/api/browser/provenance")
       .then((resp) => {
         const provenance = resp.data;
-        const provDomain = {};
+        const provenanceNames = {};
         for (const provId in provenance) {
-          const url = provenance[provId];
-          try {
-            provDomain[provId] = new URL(url).host;
-          } catch (err) {
-            console.log("Invalid url in prov: " + url);
+          const provInfo = provenance[provId];
+          if (provInfo?.name) {
+            provenanceNames[provId] = provInfo.name;
+          } else {
+            provenanceNames[provId] = provId;
           }
         }
         this.setState({
           dataFetched: true,
-          provDomain,
+          provenanceNames,
         });
       })
       .catch((e) => {

static/js/browser/arc_table_row.tsx

@@ -32,9 +32,9 @@ const NUM_VALUES_UNEXPANDED = 5;
 interface ArcTableRowPropType {
   propertyLabel: string;
   values: Array<ArcValue>;
-  // If provenanceId and src are skipped, ensure that table only has 2-columns.
+  // If provenanceId and provenanceName are skipped, ensure that table only has 2-columns.
   provenanceId?: string;
-  src?: URL;
+  provenanceName?: string;
   // If set to true, will not add a link to the property node.
   noPropLink?: boolean;
   // Index of the property label; the same property can be listed multiple times
@@ -137,11 +137,11 @@ export class ArcTableRow extends React.Component<
               })}
           </div>
         </td>
-        {this.props.provenanceId && this.props.src ? (
+        {this.props.provenanceId && this.props.provenanceName ? (
           <td className="provenance-column">
             {this.props.provenanceId && (
               <a href={HREF_PREFIX + this.props.provenanceId}>
-                {this.props.src}
+                {this.props.provenanceName}
               </a>
             )}
           </td>

static/js/browser/in_arc_section.tsx

@@ -33,7 +33,7 @@ const LOADING_CONTAINER_ID = "browser-in-arc-section";
 interface InArcSectionsPropType {
   nodeName: string;
   dcid: string;
-  provDomain: { [key: string]: URL };
+  provenanceNames: { [key: string]: string };
 }
 interface InArcSectionStateType {
   data: { [parentType: string]: { [property: string]: Array<InArcValue> } };
@@ -78,7 +78,7 @@ export class InArcSection extends React.Component<
                     parentType={parentType}
                     property={predicate}
                     arcValues={arcsByPredicate[predicate]}
-                    provDomain={this.props.provDomain}
+                    provenanceNames={this.props.provenanceNames}
                     key={parentType + predicate}
                   />
                 );

static/js/browser/in_arc_subsection.tsx

@@ -29,7 +29,7 @@ interface InArcSubsectionPropType {
   parentType: string;
   property: string;
   arcValues: Array<InArcValue>;
-  provDomain: { [key: string]: URL };
+  provenanceNames: { [key: string]: string };
 }
 
 export class InArcSubsection extends React.Component<InArcSubsectionPropType> {
@@ -73,10 +73,8 @@ export class InArcSubsection extends React.Component<InArcSubsectionPropType> {
                     propertyLabel={this.props.property}
                     values={[{ dcid: arcValue.dcid, text: valueText }]}
                     provenanceId={arcValue.provenanceId}
-                    src={
-                      this.props.provDomain[arcValue.provenanceId]
-                        ? this.props.provDomain[arcValue.provenanceId]
-                        : null
+                    provenanceName={
+                      this.props.provenanceNames[arcValue.provenanceId] || null
                     }
                   />
                 );

static/js/browser/out_arc_section.tsx

@@ -67,7 +67,7 @@ export function shouldIgnoreProperty(property: string): boolean {
 
 interface OutArcSectionPropType {
   dcid: string;
-  provDomain: { [key: string]: URL };
+  provenanceNames: { [key: string]: string };
   nodeTypes: string[];
   showAllProperties: boolean;
 }
@@ -135,7 +135,7 @@ export class OutArcSection extends React.Component<
               propertyLabel={DCID_PREDICATE}
               values={[{ text: this.props.dcid }]}
               provenanceId={""}
-              src={null}
+              provenanceName={null}
             />
             {predicates.map((predicate) => {
               const valuesByProvenance = this.state.data[predicate];
@@ -147,10 +147,8 @@ export class OutArcSection extends React.Component<
                       propertyLabel={predicate}
                       values={valuesByProvenance[provenanceId]}
                       provenanceId={provenanceId}
-                      src={
-                        this.props.provDomain[provenanceId]
-                          ? this.props.provDomain[provenanceId]
-                          : null
+                      provenanceName={
+                        this.props.provenanceNames[provenanceId] || null
                       }
                       propIndex={index}
                     />