Merge pull request #30 from launchdarkly/eb/ch24369/all-flags-caching

cache flag data in allFlags

Author: eli-darkly

src/LaunchDarkly/LDClient.php

@@ -350,14 +350,17 @@ public function allFlagsState($user, $options = array())
  return new FeatureFlagsState(false);
  }
 
+ $preloadedRequester = new PreloadedFeatureRequester($this->_featureRequester, $flags);
+ // This saves us from doing repeated queries for prerequisite flags during evaluation
+
  $state = new FeatureFlagsState(true);
  $clientOnly = isset($options['clientSideOnly']) && $options['clientSideOnly'];
  $withReasons = isset($options['withReasons']) && $options['withReasons'];
  foreach ($flags as $key => $flag) {
  if ($clientOnly && !$flag->isClientSide()) {
  continue;
  }
- $result = $flag->evaluate($user, $this->_featureRequester);
+ $result = $flag->evaluate($user, $preloadedRequester);
  $state->addFlag($flag, $result->getDetail(), $withReasons);
  }
  return $state;

src/LaunchDarkly/PreloadedFeatureRequester.php

@@ -0,0 +1,52 @@
+<?php
+namespace LaunchDarkly;
+
+class PreloadedFeatureRequester implements FeatureRequester
+{
+ /** @var FeatureRequester */
+ private $_baseRequester;
+
+ /** @var array */
+ private $_knownFeatures;
+
+ public function __construct($baseRequester, $knownFeatures)
+ {
+ $this->_baseRequester = $baseRequester;
+ $this->_knownFeatures = $knownFeatures;
+ }
+
+ /**
+ * Gets feature data from cached values
+ *
+ * @param $key string feature key
+ * @return FeatureFlag|null The decoded FeatureFlag, or null if missing
+ */
+ public function getFeature($key)
+ {
+ if (isset($this->_knownFeatures[$key])) {
+ return $this->_knownFeatures[$key];
+ }
+ return null;
+ }
+
+ /**
+ * Gets segment data from the regular feature requester
+ *
+ * @param $key string segment key
+ * @return Segment|null The decoded Segment, or null if missing
+ */
+ public function getSegment($key)
+ {
+ return $this->_baseRequester->getSegment($key);
+ }
+
+ /**
+ * Gets all features from cached values
+ *
+ * @return array()|null The decoded FeatureFlags, or null if missing
+ */
+ public function getAllFeatures()
+ {
+ return $this->_knownFeatures;
+ }
+}