The details of recording a view are hidden away inside the api call.

These details should be in a class (or set of classes) within the code base and be subject to testing with the API left to do it's job of parameter parsing and error handling.

The motivation for this refactor is the ongoing boost improvements. A critical piece of boost functionality that marks a boost as expired has been hidden away inside this seemingly unrelated API call.

Closes #1077

Testing Steps:

• Scroll down newsfeed
• Make a note of the view counters
• Wait an indeterminate amount of time (Counters are stored on the entity within ES?)
• Refresh page
• The view counters should have increased by 1

added scoped labels

changed the description

added 1 commit

• eb34ad78 - Extract record view details from analytics/views api call - #1077

Compare with previous version

added 1 commit

• cebc9837 - Extract record view details from analytics/views api call - #1077

Compare with previous version

More regression...

2019/12/04 15:36:51 [error] 9#9: *65 FastCGI sent in stderr: "PHP message: Exception #1575473811: TypeError: Return value of Minds\Core\Analytics\Views\Record::recordEntity() must be of the type bool, none returned in /var/www/Minds/engine/Core/Analytics/Views/Record.php:144
Stack trace:
#0 /var/www/Minds/engine/Controllers/api/v2/analytics/views.php(37): Minds\Core\Analytics\Views\Record->recordEntity()
#1 /var/www/Minds/engine/Api/Factory.php(87): Minds\Controllers\api\v2\analytics\views->post(Array)
#2 /var/www/Minds/engine/Controllers/api/api.php(81): Minds\Api\Factory::build(Array, Object(Zend\Diactoros\ServerRequest), Object(Zend\Diactoros\Response\JsonResponse))
#3 /var/www/Minds/engine/Core/Router.php(157): Minds\Controllers\api\api->post(Array)
#4 /var/www/Minds/engine/index.php(9): Minds\Core\Router->route()
#5 {main}" while reading response header from upstream, client: 192.168.13.110, server: _, request: "POST /api/v2/analytics/views/activity/1041704347395690512 HTTP/1.1", upstream: "fastcgi://127.0.0.1:9000", host: "extract-view-api-1077.minds.io", referrer: "https://extract-view-api-1077.minds.io/newsfeed/subscriptions"

Also...

2019/12/04 15:37:08 [error] 9#9: *65 FastCGI sent in stderr: "PHP message: Attribute rejectReason is not defined in Minds\Core\Boost\Network\Boost (setRejectReason)" while reading response header from upstream, client: 192.168.13.110, server: _, request: "POST /api/v2/analytics/views/boost/1018217353512488976 HTTP/1.1", upstream: "fastcgi://127.0.0.1:9000", host: "extract-view-api-1077.minds.io", referrer: "https://extract-view-api-1077.minds.io/newsfeed/subscriptions"

Objects of type Minds\Core\Boost\Network\Boost are what is read but the functions we pass that object to are actually expecting an object of type Minds\Entities\Boost\Network

This is a lift and shift of original functionality. We have two objects declared for the same thing??? I'm confused now.

Minds\Entities\Boost\Network has a setRejectionReason method. Somebody made a typo with setRejectReason and this has never been caught????

Should be setRejectedReason. Was being set by Boost\Network\Repository so never worked since that file was created.

added 1 commit

• 6cefab06 - Extract record view details from analytics/views api call - #1077

Compare with previous version

unmarked as a Work In Progress

Needs testing instructions. How can I verify this?

assigned to @gthouret

changed the description

changed the description

changed the description

added scoped label and automatically removed label

@gthouret this needs to be pointed at the new epic you're creating for the boost refactor test site.

added 1 commit

• 65285097 - Add phpdocs and move referral cookie instantiation to constructor - #1077

Compare with previous version

changed target branch from master to epic/boost-campaign

added scoped label and automatically removed label

@gthouret there are merge conflicts.

added scoped label and automatically removed label

added scoped label