#7197 closed defect (bug) (fixed)
404 errors when using HHVM on BP 2.6.1.1
Reported by: | r-a-y | Owned by: | r-a-y |
---|---|---|---|
Milestone: | 2.6.2 | Priority: | normal |
Severity: | normal | Version: | 2.6.0 |
Component: | Core | Keywords: | has-patch commit |
Cc: |
Description
There are reports on the forums with BP 2.6.1.1 causing 404 errors when HHVM is used:
https://buddypress.org/support/topic/page-not-found-404-when-trying-to-access-a-member-profile/#post-256546
I'll try to spin up a HHVM instance over the weekend to try and debug the problem. If anyone uses a good vagrant box for HHVM, let me know before I start looking around.
Attachments (1)
Change History (7)
This ticket was mentioned in Slack in #buddypress by mercime. View the logs.
8 years ago
#4
@
8 years ago
Won't have caught this, but i've added experimental support for HHVM into our Travis-CI configuration in r10975
Confirmed on a HHVM install. Testing on HHVM actually revealed a bug in the create_nav_link() functions during the implementation of the new nav API (#6534).
We shouldn't be returning the nav API methods, we should be returning the array items.
Attached patch works on HHVM and non-HHVM installs.