Ver Fonte

clean code

Yuki Takei há 6 anos atrás
pai
commit
e7fb97584d

+ 7 - 5
src/client/js/components/PageList/PagePathHierarchicalLink.jsx

@@ -37,13 +37,15 @@ const PagePathHierarchicalLink = (props) => {
   return (
   return (
     <>
     <>
       { isParentExists && (
       { isParentExists && (
-        <PagePathHierarchicalLink linkedPagePath={linkedPagePath.parent} />
-      ) }
-      { isParentExists && !isParentRoot && (
-        <span className="separator">/</span>
+        <>
+          <PagePathHierarchicalLink linkedPagePath={linkedPagePath.parent} />
+          { !isParentRoot && (
+            <span className="separator">/</span>
+          ) }
+        </>
       ) }
       ) }
-
       { !isParentExists && <RootLink /> }
       { !isParentExists && <RootLink /> }
+
       <a classNames={classNames} href={encodeURI(linkedPagePath.href)}>{linkedPagePath.pathName}</a>
       <a classNames={classNames} href={encodeURI(linkedPagePath.href)}>{linkedPagePath.pathName}</a>
     </>
     </>
   );
   );

+ 3 - 5
src/client/js/models/LinkedPagePath.js

@@ -2,6 +2,9 @@ import { pathUtils } from 'growi-commons';
 
 
 import PagePath from './PagePath';
 import PagePath from './PagePath';
 
 
+/**
+ * Linked Array Structured PagePath Model
+ */
 export default class LinkedPagePath {
 export default class LinkedPagePath {
 
 
   constructor(path, skipNormalize = false) {
   constructor(path, skipNormalize = false) {
@@ -23,9 +26,4 @@ export default class LinkedPagePath {
     return pathUtils.normalizePath(`${this.parent.href}/${this.pathName}`);
     return pathUtils.normalizePath(`${this.parent.href}/${this.pathName}`);
   }
   }
 
 
-  get escapedPathName() {
-    // TODO: impl
-    return this.pathName;
-  }
-
 }
 }