Skip to content

Commit

Permalink
Merged Pull Request '#39 main->main : Defer checking whether CloudReq…
Browse files Browse the repository at this point in the history
…uestEngine's data contains element's d…'

Defer checking whether CloudRequestEngine's data contains element's d…
  • Loading branch information
Automation51D authored Nov 9, 2023
2 parents 5f82aca + 736bd7a commit de8a626
Showing 1 changed file with 25 additions and 23 deletions.
48 changes: 25 additions & 23 deletions src/GeoLocationCloud.php
Original file line number Diff line number Diff line change
@@ -1,24 +1,24 @@
<?php
/* *********************************************************************
* This Original Work is copyright of 51 Degrees Mobile Experts Limited.
* Copyright 2023 51 Degrees Mobile Experts Limited, Davidson House,
* Forbury Square, Reading, Berkshire, United Kingdom RG1 3EU.
*
* This Original Work is licensed under the European Union Public Licence
* (EUPL) v.1.2 and is subject to its terms as set out below.
*
* If a copy of the EUPL was not distributed with this file, You can obtain
* one at https://opensource.org/licenses/EUPL-1.2.
*
* The 'Compatible Licences' set out in the Appendix to the EUPL (as may be
* amended by the European Commission) shall be deemed incompatible for
* the purposes of the Work and the provisions of the compatibility
* clause in Article 5 of the EUPL shall not apply.
*
* If using the Work as, or as part of, a network application, by
* including the attribution notice(s) required under Article 5 of the EUPL
* in the end user terms of the application under an appropriate heading,
* such notice(s) shall fulfill the requirements of that article.
/* *********************************************************************
* This Original Work is copyright of 51 Degrees Mobile Experts Limited.
* Copyright 2023 51 Degrees Mobile Experts Limited, Davidson House,
* Forbury Square, Reading, Berkshire, United Kingdom RG1 3EU.
*
* This Original Work is licensed under the European Union Public Licence
* (EUPL) v.1.2 and is subject to its terms as set out below.
*
* If a copy of the EUPL was not distributed with this file, You can obtain
* one at https://opensource.org/licenses/EUPL-1.2.
*
* The 'Compatible Licences' set out in the Appendix to the EUPL (as may be
* amended by the European Commission) shall be deemed incompatible for
* the purposes of the Work and the provisions of the compatibility
* clause in Article 5 of the EUPL shall not apply.
*
* If using the Work as, or as part of, a network application, by
* including the attribution notice(s) required under Article 5 of the EUPL
* in the end user terms of the application under an appropriate heading,
* such notice(s) shall fulfill the requirements of that article.
* ********************************************************************* */

namespace fiftyone\pipeline\geolocation;
Expand Down Expand Up @@ -48,10 +48,12 @@ public function __construct($settings = array("locationProvider" => "fiftyonedeg
}
}

public function onRegistration($pipeline) {
if (!array_key_exists($this->dataKey, $pipeline->flowElementsList["cloud"]->flowElementProperties)) {
public function processInternal($flowData)
{
if (!array_key_exists($this->dataKey, $flowData->pipeline->flowElementsList["cloud"]->flowElementProperties)) {
throw new \Exception("Location data was not available. Check that this key is authorised for geolocation data");
}
return parent::onRegistration($pipeline);

parent::processInternal($flowData);
}
}

0 comments on commit de8a626

Please sign in to comment.