This add-on seems to block the "replace_link_with_first_in_nav" attribute from working, at least on Concrete5 8.4.2. The redirection had suddenly stopped working, but after disabling Speed Analyzer it started to work again.

Concrete5 8.5.4 (started from 8.4.5 or 8.5.0)

An exception occurred while executing 'ALTER TABLE SpeedAnalyzerReportEvents CHANGE reportId reportId CHAR(36) DEFAULT NULL COMMENT '(DC2Type:guid)'':

SQLSTATE[HY000]: General error: 1832 Cannot change column 'reportId': used in a foreign key constraint 'FK_1B88CBECBB333E0D'

In this table I have same as above change and set NULL to yes.
So I do not now what is changing?
I assume change reportId to reportId trigger exception.

• Depending on the actual speed requirements here, I'd probably just do an insert query with my own simple value objects rather than using the ORM, since the ORM has a lot of extra considerations to make and is prone to bloating

Related class: https://github.com/a3020/speed_analyzer/blob/master/src/SpeedAnalyzer/Entity/Report.php

• This is more a service than a repository
• Be careful with FOREIGN_KEY_CHECKS configuration. It'd be better to properly manage the cascade of deletes within the system

Related class: https://github.com/a3020/speed_analyzer/blob/master/src/SpeedAnalyzer/Report/ReportRepository.php

• GeoLocationResult could be memoized at least for the server IP. It isn't likely to change

Related class: https://github.com/a3020/speed_analyzer/blob/master/src/SpeedAnalyzer/Ajax/Diagnosis/Location.php