Merge pull request #520 from trheyi/main

[add] builder page create and remove apis

sui/api/api.go

@@ -118,10 +118,10 @@ var dsl = []byte(`
 			"in": ["$param.id", "$param.template_id", "$param.route", ":context"],
 			"out": { "status": 200, "type": "application/json" }
 		},{
-			"path": "/:id/page/create/:template_id/*route",
+			"path": "/:id/page/create/:template_id",
 			"method": "POST",
 			"process": "sui.Page.Create",
-			"in": ["$param.id", "$param.template_id", "$param.route", ":context"],
+			"in": ["$param.id", "$param.template_id", ":payload", ":context"],
 			"out": { "status": 200, "type": "application/json" }
 		},{
 			"path": "/:id/page/exist/:template_id/*route",

sui/api/process.go

@@ -457,7 +457,7 @@ func PageSave(process *process.Process) interface{} {
 			exception.New(err.Error(), 500).Throw()
 		}
 	} else {
-		page, err = tmpl.CreatePage(route)
+		page, err = tmpl.CreateEmptyPage(route, nil)
 		if err != nil {
 			exception.New(err.Error(), 500).Throw()
 		}
@@ -499,7 +499,7 @@ func PageSaveTemp(process *process.Process) interface{} {
 			exception.New(err.Error(), 500).Throw()
 		}
 	} else {
-		page, err = tmpl.CreatePage(route)
+		page, err = tmpl.CreateEmptyPage(route, nil)
 		if err != nil {
 			exception.New(err.Error(), 500).Throw()
 		}
@@ -530,14 +530,23 @@ func PageCreate(process *process.Process) interface{} {
 	process.ValidateArgNums(3)
 	sui := get(process)
 	templateID := process.ArgsString(1)
-	route := route(process, 2)
+	payload := process.ArgsMap(2, map[string]interface{}{})
 
 	tmpl, err := sui.GetTemplate(templateID)
 	if err != nil {
 		exception.New(err.Error(), 500).Throw()
 	}
 
-	page, err := tmpl.CreatePage(route)
+	route, ok := payload["route"].(string)
+	if !ok {
+		exception.New("the route is required", 400).Throw()
+	}
+	title := route
+	if v, ok := payload["title"].(string); ok {
+		title = v
+	}
+	setting := &core.PageSetting{Title: title}
+	page, err := tmpl.CreateEmptyPage(route, setting)
 	if err != nil {
 		exception.New(err.Error(), 500).Throw()
 	}
@@ -548,7 +557,7 @@ func PageCreate(process *process.Process) interface{} {
 
 	source, err := getSource(process)
 	if err != nil {
-		exception.New(err.Error(), 500).Throw()
+		return nil
 	}
 
 	if source == nil {
@@ -574,6 +583,10 @@ func PageRemove(process *process.Process) interface{} {
 		exception.New(err.Error(), 500).Throw()
 	}
 
+	if !tmpl.PageExist(route) {
+		exception.New("page does not exists!", 400).Throw()
+	}
+
 	err = tmpl.RemovePage(route)
 	if err != nil {
 		exception.New(err.Error(), 500).Throw()

sui/core/interfaces.go

@@ -37,7 +37,7 @@ type ITemplate interface {
 	PageTree(route string) ([]*PageTreeNode, error)
 	Page(route string) (IPage, error)
 	PageExist(route string) bool
-	CreatePage(route string) (IPage, error)
+	CreateEmptyPage(route string, setting *PageSetting) (IPage, error)
 	RemovePage(route string) error
 	GetPageFromAsset(asset string) (IPage, error)
 

sui/core/types.go

@@ -172,7 +172,11 @@ type RequestSource struct {
 	Board      *BoardSourceData        `json:"board,omitempty"`
 	Mock       *PageMock               `json:"mock,omitempty"`
 	Setting    *PageSetting            `json:"setting,omitempty"`
-	NeedToSave struct {
+	NeedToSave ReqeustSourceNeedToSave `json:"needToSave,omitempty"`
+}
+
+// ReqeustSourceNeedToSave is the struct for the request
+type ReqeustSourceNeedToSave struct {
 	Page     bool `json:"page,omitempty"`
 	Style    bool `json:"style,omitempty"`
 	Script   bool `json:"script,omitempty"`
@@ -181,7 +185,6 @@ type RequestSource struct {
 	Mock     bool `json:"mock,omitempty"`
 	Setting  bool `json:"setting,omitempty"`
 	Validate bool `json:"validate,omitempty"`
-	} `json:"needToSave,omitempty"`
 }
 
 // ResponseEditorRender is the struct for the response

sui/storages/local/page.go

@@ -186,6 +186,8 @@ func (tmpl *Template) RemovePage(route string) error {
 
 	path := filepath.Join(tmpl.Root, route)
 	name := filepath.Base(path) + ".*"
+	name = strings.ReplaceAll(name, "[", "\\[")
+	name = strings.ReplaceAll(name, "]", "\\]")
 	err := tmpl.local.fs.Walk(path, func(root, file string, isdir bool) error {
 		if isdir {
 			return nil
@@ -210,15 +212,15 @@ func (tmpl *Template) RemovePage(route string) error {
 
 }
 
-// CreatePage create a new page
+// CreateEmptyPage create a new empty
-func (tmpl *Template) CreatePage(route string) (core.IPage, error) {
+func (tmpl *Template) CreateEmptyPage(route string, setting *core.PageSetting) (core.IPage, error) {
 	if tmpl.PageExist(route) {
 		return nil, fmt.Errorf("Page %s already exist", route)
 	}
 
 	// Create the page directory
 	name := tmpl.getPageBase(route)
-	return &Page{
+	page := &Page{
 		tmpl: tmpl,
 		Page: &core.Page{
 			Route:      route,
@@ -235,7 +237,22 @@ func (tmpl *Template) CreatePage(route string) (core.IPage, error) {
 				CONF: core.Source{File: fmt.Sprintf("%s.config", name)},
 			},
 		},
-	}, nil
+	}
+
+	title := route
+	if setting != nil {
+		title = setting.Title
+	}
+
+	err := page.Save(&core.RequestSource{
+		Page:       &core.SourceData{Source: fmt.Sprintf("<div>%s</div>", title), Language: "html"},
+		Setting:    setting,
+		NeedToSave: core.ReqeustSourceNeedToSave{Page: true, Setting: true},
+	})
+	if err != nil {
+		return nil, err
+	}
+	return page, nil
 }
 
 // Remove remove the page

sui/storages/local/page_test.go

@@ -474,7 +474,7 @@ func TestPageSave(t *testing.T) {
 	req := &core.RequestSource{UID: "19e09e7e-9e19-44c1-bbab-2a55c51c9df3"}
 	jsoniter.Unmarshal([]byte(payload), &req)
 
-	page, err := tmpl.CreatePage("/unit-test")
+	page, err := tmpl.CreateEmptyPage("/unit-test", nil)
 	if err != nil {
 		t.Fatalf("Page error: %v", err)
 	}