Updated TODOs

This commit is contained in:
Spencer
2018-07-10 16:46:51 -04:00
parent 4294932b1f
commit f239907bb2
3 changed files with 6 additions and 8 deletions

View File

@@ -38,13 +38,11 @@ $ go get -u github.com/sbrow/ps
`sbrow:` (2) Make TextLayer a subclass of ArtLayer. `sbrow:` (2) Make TextLayer a subclass of ArtLayer.
`sbrow:` Reduce cylcomatic complexity `sbrow:` Reduce cylcomatic complexity of ActiveDocument().
`sbrow:` Does funky things when passed invalid layername. `sbrow:` refactor Close to Document.Close
`sbrow:` Move to Document `sbrow:` get rid of the semicolon at the end of JSLayer.
`sbrow:` get rid of the semicolon.
## <a name="pkg-doc">Documentation</a> ## <a name="pkg-doc">Documentation</a>
For full Documentation please visit https://godoc.org/github.com/sbrow/ps For full Documentation please visit https://godoc.org/github.com/sbrow/ps

View File

@@ -116,7 +116,7 @@ func (d *Document) LayerSet(name string) *LayerSet {
// ActiveDocument returns document currently focused in Photoshop. // ActiveDocument returns document currently focused in Photoshop.
// //
// TODO(sbrow): Reduce cylcomatic complexity // TODO(sbrow): Reduce cylcomatic complexity of ActiveDocument().
func ActiveDocument() (*Document, error) { func ActiveDocument() (*Document, error) {
log.Println("Loading ActiveDoucment") log.Println("Loading ActiveDoucment")
d := &Document{} d := &Document{}

4
ps.go
View File

@@ -40,7 +40,7 @@ func ApplyDataset(name string) error {
} }
// Close closes the active document in Photoshop, using the given save option. // Close closes the active document in Photoshop, using the given save option.
// TODO(sbrow): Move to Document // TODO(sbrow): refactor Close to Document.Close
func Close(save SaveOption) error { func Close(save SaveOption) error {
_, err := runner.Run("close", fmt.Sprint(save)) _, err := runner.Run("close", fmt.Sprint(save))
return err return err
@@ -106,7 +106,7 @@ func Init() error {
// The output always ends with a semicolon, so if you want to access a specific // The output always ends with a semicolon, so if you want to access a specific
// property of the layer, you'll have to trim the output before concatenating. // property of the layer, you'll have to trim the output before concatenating.
// //
// TODO(sbrow): get rid of the semicolon. // TODO(sbrow): get rid of the semicolon at the end of JSLayer.
func JSLayer(path string) string { func JSLayer(path string) string {
pth := strings.Split(path, "/") pth := strings.Split(path, "/")
js := "app.activeDocument" js := "app.activeDocument"