Reflecting krapie's comment Changing mismatched code conventions and …

…rollback to String "null"

pkg/document/crdt/primitive.go

@@ -196,7 +196,7 @@ func (p *Primitive) Bytes() []byte {
 func (p *Primitive) Marshal() string {
 	switch p.valueType {
 	case Null:
-		return ""
+		return "null"
 	case Boolean:
 		return fmt.Sprintf("%t", p.value)
 	case Integer:
@@ -213,7 +213,6 @@ func (p *Primitive) Marshal() string {
 		return fmt.Sprintf(`"%s"`, p.value)
 	case Date:
 		return fmt.Sprintf(`"%s"`, p.value.(gotime.Time).Format(gotime.RFC3339))
-		// when Null or default
 	default:
 		return ""
 	}

pkg/document/crdt/primitive_test.go

@@ -34,7 +34,7 @@ func TestPrimitive(t *testing.T) {
 		valueType crdt.ValueType
 		marshal   string
 	}{
-		{nil, crdt.Null, ""},
+		{nil, crdt.Null, "null"},
 		{false, crdt.Boolean, "false"},
 		{true, crdt.Boolean, "true"},
 		{0, crdt.Integer, "0"},

pkg/document/json/array.go

@@ -227,6 +227,7 @@ func (p *Array) addInternal(
 ) crdt.Element {
 	return p.insertAfterInternal(p.Array.LastCreatedAt(), creator)
 }
+
 func (p *Array) insertAfterInternal(
 	prevCreatedAt *time.Ticket,
 	creator func(ticket *time.Ticket) crdt.Element,

pkg/document/json/object.go

@@ -130,6 +130,7 @@ func (p *Object) SetNull(k string) *Object {
 		}
 		return primitive
 	})
+
 	return p
 }
 
@@ -142,6 +143,7 @@ func (p *Object) SetBool(k string, v bool) *Object {
 		}
 		return primitive
 	})
+
 	return p
 }
 
@@ -154,6 +156,7 @@ func (p *Object) SetInteger(k string, v int) *Object {
 		}
 		return primitive
 	})
+
 	return p
 }
 
@@ -179,6 +182,7 @@ func (p *Object) SetDouble(k string, v float64) *Object {
 		}
 		return primitive
 	})
+
 	return p
 }