Syracuse, NY
Account created on 2 March 2013, over 11 years ago
#

Recent comments

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

I've not observed any further occurrences of this issue in my testing. I believe the underlying APIs have been updated in drupal 9 to correct this. Marking as outdated, but can be re-opened if it's spotted again.

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

I'll review and include if appropriate in the next release. Since changing the format of a readme document provides little to no value this can not be a priority over other issues in the queue.

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

I'm not able to reproduce this issue, it's possible it does not apply to the current version of simple_access_log and Drupal 9+

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

Fixed and released with version 2.0

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

This may be possible for core entity paths such as Nodes and Taxonomy, but with overridable paths and custom entities from contributed modules I can see this as being a difficult thing to implement well. That being said I do see value to being able to use the logs captured in simple_access_log being related to nodes in particular for the purposes of gauging popularity/visit frequency of particular nodes.

I'd gladly accept any patches if someone is interested in contributing code to this effort.

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

Removed caching on default view in version 2.0.1

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

This has been released as 2.0.1

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

I've added this to the 2.0.0 release today. I needed to modify the submitted patch slightly, as written it would have dropped existing configurations and changed default behavior on upgrade. This change keeps the paths already entered in the configurations with exclude as the default behavior, but can be toggled to include paths only from the existing configuration page.

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

Fixed @ beta4

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

Thank you for the Patch! I'll review and test this weekend. I'm glad to see that people are finding this module helpful still, as I've been neglecting it lately.

πŸ‡ΊπŸ‡ΈUnited States benjaminarthurt Syracuse, NY

#26 works against the current Drupal 10.0.2 w/ php 8.2.1 Now that D10 is no longer a beta release it would be great to get this into patch committed.

Production build 0.69.0 2024