pointers: rename Eq() to Equal()

[m940867]

What does this PR do?

https://github.com/upfluence/ecommerce-server/pull/309#discussion_r2651213243

What are the observable changes?

Good PR checklist

• Title makes sense
• Is against the correct branch
• Only addresses one issue
• Properly assigned
• Added/updated tests
• Added/updated documentation
• Properly labeled

Additional Notes

---

Note

Rename and enhance equality helper

• Renames Eq[T] to Equal[T] across code and tests
• Equal now defers to a type’s own Equal(b T) bool method when implemented (via interface check), otherwise falls back to *a == *b
• Tests updated to cover renamed API and new behavior, including time.Time comparisons across zones

^{Written by Cursor Bugbot for commit 5c6663e. This will update automatically on new commits. Configure here.}

[AlexisMontagne]

what would you think of adding smth like:

       if aa, ok := a.(interface{ Equal(b T) bool }); ok {
               return aa.Equal(*b)
       }

before the final return?

[m940867]

This doesn't compile for me:

pointers/ptr.go:35:15: invalid operation: a (variable of type *T) is not an interface

[AlexisMontagne]

this is a bummer :(. tho i do have a dirty trick for you. you can add a var v any = a and do the type assertion on v.

[m940867]

Ah yes, thank you

[AlexisMontagne]

your syntax is much better 👍🏼

can i ask you to add a test to assert this? you could use some *time.Time for example

[m940867]

Sure

[cursor]

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.