/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package io.servicecomb.transport.rest.vertx.accesslog.element.impl;

import io.servicecomb.transport.rest.vertx.accesslog.AccessLogParam;
import io.servicecomb.transport.rest.vertx.accesslog.element.AccessLogElement;
import io.vertx.core.http.HttpServerResponse;

/**
 * Zero bytes written as -
 */
public class BytesWrittenV2Element implements AccessLogElement {

  public static final String ZERO_BYTES = "-";

  @Override
  public String getFormattedElement(AccessLogParam accessLogParam) {
    HttpServerResponse response = accessLogParam.getRoutingContext().response();
    if (null == response) {
      return ZERO_BYTES;
    }

    long bytesWritten = response.bytesWritten();
    if (0 == bytesWritten) {
      return ZERO_BYTES;
    } else {
      return String.valueOf(bytesWritten);
    }
  }
}

@@ -1543,18 +1543,10 @@
    %7D
- else %7B%0A  
+%0A%0A
     
@@ -1582,20 +1582,14 @@
 itten);%0A
-    %7D%0A
   %7D%0A%7D%0A

remove redundant else expression