revert: CCS 3861 , hold for release (#527)

pantheon-bundle/frontend/src/app/__snapshots__/contentDisplay.test.tsx.snap

@@ -158,8 +158,8 @@ exports[`ContentDisplay tests for Assembly should render ContentDisplay componen
       contentType="module"
       modulePath=""
       onGetProduct={[Function]}
-      onGetUrl={[Function]}
       onGetVersion={[Function]}
+      onPublishEvent={[Function]}
       productInfo=""
       updateDate={[Function]}
       variant="DEFAULT"
@@ -338,8 +338,8 @@ exports[`ContentDisplay tests for Module should render ModuleDisplay component 1
       contentType="module"
       modulePath=""
       onGetProduct={[Function]}
-      onGetUrl={[Function]}
       onGetVersion={[Function]}
+      onPublishEvent={[Function]}
       productInfo=""
       updateDate={[Function]}
       variant="DEFAULT"

pantheon-bundle/frontend/src/app/contentDisplay.tsx

@@ -214,9 +214,9 @@ class ContentDisplay extends Component<any, IModuleDisplayState | IAssemblyDispl
                         attributesFilePath={this.state.attributesFilePath}
                         assemblies={this.state.assemblyData}
                         updateDate={this.updateDate}
-                        onGetUrl={this.onGetUrl}
                         onGetProduct={this.getProduct}
                         onGetVersion={this.getVersion}
+                        onPublishEvent={this.onPublishEvent}
                     />
                 </Card>
 
@@ -238,8 +238,8 @@ class ContentDisplay extends Component<any, IModuleDisplayState | IAssemblyDispl
             releaseVersion,
         })
     }
-    
-    
+
+
     private fetchModuleDetails = async (data) => {
         console.log(data)
         await this.getVariantParam()
@@ -324,11 +324,11 @@ class ContentDisplay extends Component<any, IModuleDisplayState | IAssemblyDispl
         this.setState({ versionValue: version })
     }
 
-    private onGetUrl = (url) => {
-        console.log("Received url="+url)
-        if(this.state.productUrlFragment!==""){
-            this.setState({ portalUrl: url})
-        }
+    private onPublishEvent = () => {
+        // the published state cannot be ascertained correctly when moving from one page to another
+        const path = this.props.location.pathname.substring(this.isAssembly ? PathPrefixes.ASSEBMLY_PATH_PREFIX.length : PathPrefixes.MODULE_PATH_PREFIX.length)
+        this.getPortalUrl(path, this.state.variant)
+
     }
 
     private getLocale = (path) =>{
@@ -407,6 +407,13 @@ class ContentDisplay extends Component<any, IModuleDisplayState | IAssemblyDispl
                             for (const productVersion of productChild.__children__) {
                                 if (productVersion[Fields.JCR_UUID] === uuid) {
                                     this.setState({ productValue: product.name, versionValue: productVersion.name, productUrlFragment: product.urlFragment, versionUrlFragment: productVersion.urlFragment })
+                                    const isGuideOrTopic = this.isAssembly ? '/guide/' : '/topic/'
+                                    const url = this.state.portalHostUrl + '/documentation/'+this.state.locale.toLocaleLowerCase()+'/' + this.state.productUrlFragment + '/' + this.state.versionUrlFragment + isGuideOrTopic + this.state.variantUUID
+                                    console.log("Constructed url="+url)
+                                    if(this.state.productUrlFragment!==""){
+                                        this.setState({ portalUrl: url})
+                                    }
+                                    break
                                 }
                             }
                         }
@@ -441,8 +448,15 @@ class ContentDisplay extends Component<any, IModuleDisplayState | IAssemblyDispl
                         // get portal url from api and set it only if it is not empty
                         if(text.trim() !== "") {
                             this.setState({portalUrl: text})
+                        }else{
+                            // if portal url is empty, assemble the URL at client side
+                            console.log("GetPortalURI API returned empty URI. Falling back to url construction at UI")
+                            this.getVersionUUID(this.props.location.pathname)
                         }
                     })
+                }else {
+                    console.log("GetPortalURI API returned error. Falling back to url construction at UI")
+                    this.getVersionUUID(this.props.location.pathname)
                 }
             })
     }

pantheon-bundle/frontend/src/app/versions.test.tsx

@@ -20,12 +20,12 @@ const props = {
     modulePath: "/modules/test",
     onGetProduct: (productValue) => anymatch,
     onGetVersion: (versionValue) => anymatch,
-    onGetUrl: (url) => anymatch,
     productInfo: "Red Hat Enterprise Linux",
     updateDate: (draftUpdateDate, releaseUpdateDate, releaseVersion, variantUUID) => anymatch,
     variant: "test",
     variantUUID: "abcd-1234",
     versionModulePath: "/modules/test_module/en_US/variants/test/draft",
+    onPublishEvent: () => anymatch
 }
 
 describe("Versions tests", () => {
@@ -193,12 +193,12 @@ describe("Versions tests", () => {
             modulePath: "somePath",
             onGetProduct: (productValue) => anymatch,
             onGetVersion: (versionValue) => anymatch,
-            onGetUrl: (url) => anymatch,
             productInfo: "Red Hat Enterprise Linux",
             updateDate: (draftUpdateDate, releaseUpdateDate, releaseVersion, variantUUID) => anymatch,
             variant: "DEFAULT",
             variantUUID: "abcd-1234",
             versionModulePath: "versionPath",
+            onPublishEvent: () => anymatch
         }
         state.updateDate("-", "-", 1, "1234")
         expect(state.modulePath).toEqual("somePath")
@@ -277,4 +277,4 @@ describe("Versions tests", () => {
         inst.setAlertTitle()
         sinon.assert.called(spy)
     })
-})
+})

pantheon-bundle/frontend/src/app/versions.tsx

@@ -28,10 +28,10 @@ export interface IProps {
     variantUUID: string
     attributesFilePath: string
     assemblies?: any
-    onGetUrl: (url) => any
     updateDate: (draftUpdateDate, releaseUpdateDate, releaseVersion, variantUUID) => any
     onGetProduct: (productValue) => any
     onGetVersion: (versionValue) => any
+    onPublishEvent: () => void
 }
 
 // Define properties in Metadata
@@ -511,11 +511,13 @@ class Versions extends Component<IProps, IState> {
                     // console.log("Published file path:", this.props.modulePath)
                     this.draft[0].version = "";
                     this.setState({ unpublishAlertForModuleVisible: false })
+                    this.props.onPublishEvent()
                 } else {
                     formData.append(":operation", "pant:unpublish");
                     // console.log("Unpublished file path:", this.props.modulePath);
                     this.release[0].version = "";
                     this.setState({ unpublishAlertForModuleVisible: true })
+                    this.props.onPublishEvent()
                 }
                 const hdrs = {
                     "Accept": "application/json",
@@ -530,13 +532,12 @@ class Versions extends Component<IProps, IState> {
                 }).then(response => {
                     if (response.status === 201 || response.status === 200) {
                         console.log(buttonText + " works: " + response.status)
-                        console.log(" response is : " + response.url)
-                        this.props.onGetUrl(response.url)
                         this.setState({
                             canChangePublishState: true,
                             publishAlertVisible: false,
                             showMetadataAlertIcon: false
                         })
+                        this.props.onPublishEvent()
                     } else {
                         console.log(buttonText + " failed " + response.status)
                         this.setState({ publishAlertVisible: true })