Просмотр исходного кода

handle edge case when filename includes Procfile as substring

sunguroku 5 лет назад
Родитель
Сommit
320ca39c96
1 измененных файлов с 4 добавлено и 2 удалено
  1. 4 2
      dashboard/src/components/repo-selector/ContentsList.tsx

+ 4 - 2
dashboard/src/components/repo-selector/ContentsList.tsx

@@ -197,7 +197,7 @@ export default class ContentsList extends Component<PropsType, StateType> {
       if (fileName.includes("Dockerfile")) {
         dockerfiles.push(fileName);
       }
-      if (this.state.currentDir === "" && fileName.includes("Procfile")) {
+      if (this.state.currentDir === "" && fileName == "Procfile") {
         this.props.setProcfilePath("./Procfile");
       }
     });
@@ -213,8 +213,9 @@ export default class ContentsList extends Component<PropsType, StateType> {
   };
 
   renderOverlay = () => {
+    console.log(this.props.procfilePath)
     if (this.props.procfilePath) {
-      let processes = Object.keys(this.state.processes);
+      let processes = this.state.processes ? Object.keys(this.state.processes) : [];
       return (
         <Overlay>
           <BgOverlay
@@ -293,6 +294,7 @@ export default class ContentsList extends Component<PropsType, StateType> {
                 this.state.processes &&
                 Object.keys(this.state.processes).length > 0
               ) {
+                console.log('setting procfile')
                 this.props.setProcfilePath("./Procfile");
               }
             }}