get_tree json fixes

- null pointer check against container names
- use actual json null values instead of "null" strings
- add "visible" property to all containers
This commit is contained in:
Zandr Martin 2016-07-05 07:23:44 -05:00
parent 58804a044f
commit c65d6e6e95
No known key found for this signature in database
GPG Key ID: AA2BB8EF77F7BBDC

View File

@ -110,27 +110,28 @@ static void ipc_json_describe_output(swayc_t *output, json_object *object) {
static void ipc_json_describe_workspace(swayc_t *workspace, json_object *object) { static void ipc_json_describe_workspace(swayc_t *workspace, json_object *object) {
int num = (isdigit(workspace->name[0])) ? atoi(workspace->name) : -1; int num = (isdigit(workspace->name[0])) ? atoi(workspace->name) : -1;
bool focused = root_container.focused == workspace->parent && workspace->parent->focused == workspace; bool focused = root_container.focused == workspace->parent && workspace->parent->focused == workspace;
const char *layout = ipc_json_layout_description(workspace);
json_object_object_add(object, "num", json_object_new_int(num)); json_object_object_add(object, "num", json_object_new_int(num));
json_object_object_add(object, "visible", json_object_new_boolean(workspace->visible));
json_object_object_add(object, "focused", json_object_new_boolean(focused)); json_object_object_add(object, "focused", json_object_new_boolean(focused));
json_object_object_add(object, "output", json_object_new_string((workspace->parent) ? workspace->parent->name : "null")); json_object_object_add(object, "output", (workspace->parent) ? json_object_new_string(workspace->parent->name) : NULL);
json_object_object_add(object, "urgent", json_object_new_boolean(false)); json_object_object_add(object, "urgent", json_object_new_boolean(false));
json_object_object_add(object, "type", json_object_new_string("workspace")); json_object_object_add(object, "type", json_object_new_string("workspace"));
json_object_object_add(object, "layout", json_object_new_string(ipc_json_layout_description(workspace))); json_object_object_add(object, "layout", (strcmp(layout, "null") == 0) ? NULL : json_object_new_string(layout));
} }
static void ipc_json_describe_view(swayc_t *view, json_object *object) { static void ipc_json_describe_view(swayc_t *view, json_object *object) {
float percent = ipc_json_child_percentage(view); float percent = ipc_json_child_percentage(view);
const char *layout = ipc_json_layout_description(view);
json_object_object_add(object, "border", json_object_new_string(ipc_json_border_description(view))); json_object_object_add(object, "border", json_object_new_string(ipc_json_border_description(view)));
json_object_object_add(object, "current_border_width", json_object_new_int(view->border_thickness)); json_object_object_add(object, "current_border_width", json_object_new_int(view->border_thickness));
json_object_object_add(object, "percent", (percent > 0) ? json_object_new_double(percent) : json_object_new_string("null")); json_object_object_add(object, "percent", (percent > 0) ? json_object_new_double(percent) : NULL);
// TODO: make urgency actually work once Sway supports it // TODO: make urgency actually work once Sway supports it
json_object_object_add(object, "urgent", json_object_new_boolean(false)); json_object_object_add(object, "urgent", json_object_new_boolean(false));
json_object_object_add(object, "focused", json_object_new_boolean(view->is_focused)); json_object_object_add(object, "focused", json_object_new_boolean(view->is_focused));
json_object_object_add(object, "type", json_object_new_string((view->is_floating) ? "floating_con" : "con")); json_object_object_add(object, "type", json_object_new_string((view->is_floating) ? "floating_con" : "con"));
json_object_object_add(object, "layout", json_object_new_string(ipc_json_layout_description(view))); json_object_object_add(object, "layout", (strcmp(layout, "null") == 0) ? NULL : json_object_new_string(layout));
if (view->class) { if (view->class) {
json_object_object_add(object, "class", json_object_new_string(view->class)); json_object_object_add(object, "class", json_object_new_string(view->class));
@ -149,8 +150,9 @@ json_object *ipc_json_describe_container(swayc_t *c) {
json_object *object = json_object_new_object(); json_object *object = json_object_new_object();
json_object_object_add(object, "id", json_object_new_int((int64_t)&c)); json_object_object_add(object, "id", json_object_new_int((int64_t)&c));
json_object_object_add(object, "name", json_object_new_string(c->name)); json_object_object_add(object, "name", (c->name) ? json_object_new_string(c->name) : NULL);
json_object_object_add(object, "rect", ipc_json_create_rect(c)); json_object_object_add(object, "rect", ipc_json_create_rect(c));
json_object_object_add(object, "visible", json_object_new_boolean(c->visible));
switch (c->type) { switch (c->type) {
case C_ROOT: case C_ROOT:
@ -288,6 +290,7 @@ json_object *ipc_json_describe_container_recursive(swayc_t *c) {
json_object *object = ipc_json_describe_container(c); json_object *object = ipc_json_describe_container(c);
int i; int i;
if (c->type != C_VIEW) {
json_object *floating = json_object_new_array(); json_object *floating = json_object_new_array();
if (c->floating && c->floating->length > 0) { if (c->floating && c->floating->length > 0) {
for (i = 0; i < c->floating->length; ++i) { for (i = 0; i < c->floating->length; ++i) {
@ -311,6 +314,7 @@ json_object *ipc_json_describe_container_recursive(swayc_t *c) {
} }
} }
json_object_object_add(object, "unmanaged_nodes", unmanaged); json_object_object_add(object, "unmanaged_nodes", unmanaged);
}
if (c->type == C_ROOT) { if (c->type == C_ROOT) {
json_object *scratchpad_json = json_object_new_array(); json_object *scratchpad_json = json_object_new_array();