single vs recursive for dirspath and filespath. readme update

Author: jeevatkm

README.md

@@ -1,6 +1,7 @@
 # essentials - aah framework
 
-[![Build Status](https://travis-ci.org/go-aah/essentials.svg?branch=master)](https://travis-ci.org/go-aah/essentials) [![codecov](https://codecov.io/gh/go-aah/essentials/branch/master/graph/badge.svg)](https://codecov.io/gh/go-aah/essentials/branch/master) [![Go Report Card](https://goreportcard.com/badge/aahframework.org/essentials)](https://goreportcard.com/report/aahframework.org/essentials) [![GoDoc](https://godoc.org/aahframework.org/essentials?status.svg)](https://godoc.org/aahframework.org/essentials)  [![License](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
+[![Build Status](https://travis-ci.org/go-aah/essentials.svg?branch=master)](https://travis-ci.org/go-aah/essentials) [![codecov](https://codecov.io/gh/go-aah/essentials/branch/master/graph/badge.svg)](https://codecov.io/gh/go-aah/essentials/branch/master) [![Go Report Card](https://goreportcard.com/badge/aahframework.org/essentials)](https://goreportcard.com/report/aahframework.org/essentials)
+[![Version](https://img.shields.io/badge/version-0.1-blue.svg)](https://github.com/go-aah/essentials/releases/latest) [![GoDoc](https://godoc.org/aahframework.org/essentials?status.svg)](https://godoc.org/aahframework.org/essentials)  [![License](https://img.shields.io/github/license/go-aah/essentials.svg)](LICENSE)
 
 ***v0.1 [released](https://github.com/go-aah/essentials/releases/latest) and tagged on Jan 22, 2017***
 
@@ -11,6 +12,7 @@
 * os
 * reflect
 * string
+* archive (zip)
 
 *`essentials` developed for aah framework. However, it's an independent library, can be used separately with any `Go` language project. Feel free to use it.*
 

filepath.go

@@ -8,6 +8,7 @@ import (
 	"bytes"
 	"fmt"
 	"io"
+	"io/ioutil"
 	"os"
 	"path/filepath"
 	"strings"
@@ -267,24 +268,56 @@ func DeleteFiles(files ...string) (errs []error) {
 }
 
 // DirsPath method returns all directories absolute path from given base path recursively.
-func DirsPath(basePath string) (pdirs []string, err error) {
-	err = Walk(basePath, func(srcPath string, info os.FileInfo, err error) error {
-		if info.IsDir() {
-			pdirs = append(pdirs, srcPath)
+func DirsPath(basePath string, recursive bool) (pdirs []string, err error) {
+	if recursive {
+		err = Walk(basePath, func(srcPath string, info os.FileInfo, err error) error {
+			if info.IsDir() {
+				pdirs = append(pdirs, srcPath)
+			}
+			return nil
+		})
+		return
+	}
+
+	var list []os.FileInfo
+	list, err = ioutil.ReadDir(basePath)
+	if err != nil {
+		return
+	}
+
+	for _, v := range list {
+		if v.IsDir() {
+			pdirs = append(pdirs, filepath.Join(basePath, v.Name()))
 		}
-		return nil
-	})
+	}
+
 	return
 }
 
 // FilesPath method returns all files absolute path from given base path recursively.
-func FilesPath(basePath string) (files []string, err error) {
-	err = Walk(basePath, func(srcPath string, info os.FileInfo, err error) error {
-		if !info.IsDir() {
-			files = append(files, srcPath)
+func FilesPath(basePath string, recursive bool) (files []string, err error) {
+	if recursive {
+		err = Walk(basePath, func(srcPath string, info os.FileInfo, err error) error {
+			if !info.IsDir() {
+				files = append(files, srcPath)
+			}
+			return nil
+		})
+		return
+	}
+
+	var list []os.FileInfo
+	list, err = ioutil.ReadDir(basePath)
+	if err != nil {
+		return
+	}
+
+	for _, v := range list {
+		if !v.IsDir() {
+			files = append(files, filepath.Join(basePath, v.Name()))
 		}
-		return nil
-	})
+	}
+
 	return
 }
 
@@ -294,5 +327,11 @@ func StripExt(name string) string {
 	if IsStrEmpty(name) {
 		return name
 	}
-	return name[:strings.IndexByte(name, '.')]
+
+	idx := strings.IndexByte(name, '.')
+	if idx > 0 {
+		return name[:idx]
+	}
+
+	return name
 }

filepath_test.go

@@ -235,12 +235,18 @@ func TestDirPaths(t *testing.T) {
 	_ = MkDirAll(path21, 0755)
 	_ = MkDirAll(path22, 0755)
 
-	dirs, err := DirsPath(join(testdataPath, "dirpaths"))
+	dirs, err := DirsPath(join(testdataPath, "dirpaths"), true)
 	assert.FailOnError(t, err, "unable to get directory list")
 	assert.True(t, IsSliceContainsString(dirs, path1))
 	assert.True(t, IsSliceContainsString(dirs, path11))
 	assert.True(t, IsSliceContainsString(dirs, path12))
 	assert.True(t, IsSliceContainsString(dirs, path21))
 	assert.True(t, IsSliceContainsString(dirs, path22))
 	assert.False(t, IsSliceContainsString(dirs, join(path22, "not-exists")))
+
+	dirs, err = DirsPath(join(testdataPath, "dirpaths", "level1"), false)
+	assert.FailOnError(t, err, "unable to get directory list")
+	assert.True(t, IsSliceContainsString(dirs, path11))
+	assert.True(t, IsSliceContainsString(dirs, path12))
+	assert.False(t, IsSliceContainsString(dirs, join(path22, "not-exists")))
 }