carno-php / tracing

src/Chips/ContextOperator.php

<?php
/**
 * Context generator for carrier
 * User: moyo
 * Date: 23/11/2017
 * Time: 4:47 PM
 */

namespace Carno\Tracing\Chips;

use Carno\Tracing\Contracts\Vars\CTX;
use Carno\Tracing\Standard\Context;
use OpenTracing\Exceptions\SpanContextNotFound;
use OpenTracing\SpanContext;

trait ContextOperator
{
    use IDGenerator;

    /**
     * @param SpanContext|null $in
     * @return SpanContext
     */
    public function fusedContext(SpanContext $in = null) : SpanContext
    {
        return $in ? $this->initFromContext($in) : $this->toNewContext();
    }

    /**
     * @return SpanContext
     */
    protected function toNewContext() : SpanContext
    {
        if (empty($traceID = $this->getTraceID())) {

It seems like getTraceID() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            throw SpanContextNotFound::create();
        }

        $overrides = [
            CTX::TRACE_ID => $traceID,
            CTX::SPAN_ID => $this->newSpanID(),
            CTX::PARENT_SPAN_ID => $this->getSpanID(),

It seems like getSpanID() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            CTX::TRACE_SAMPLED => $this->checkIsSampled(),

It seems like checkIsSampled() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        ];

        return new Context(array_merge($this->getBaggageItems(), $overrides));

It seems like getBaggageItems() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param SpanContext $ctx
     * @return SpanContext
     */
    protected function initFromContext(SpanContext $ctx) : SpanContext
    {
        $this->setTraceID($ctx[CTX::TRACE_ID] ?? '');

It seems like setTraceID() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->setSpanID($ctx[CTX::SPAN_ID] ?? '');

It seems like setSpanID() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($ctx[CTX::PARENT_SPAN_ID] ?? '') {
            $this->setParentSpanID($ctx[CTX::PARENT_SPAN_ID]);

It seems like setParentSpanID() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        $this->markIsSampled($ctx[CTX::TRACE_SAMPLED] ?? false);

It seems like markIsSampled() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $items = [];

        foreach ($ctx->getIterator() as $k => $v) {
            if (in_array($k, [CTX::TRACE_ID, CTX::SPAN_ID, CTX::PARENT_SPAN_ID, CTX::TRACE_SAMPLED])) {
                continue;
            } else {
                $items[$k] = $v;
            }
        }

        $items && $this->setBaggageItems($items);

It seems like setBaggageItems() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $ctx;
    }
}