New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP Notice: Trying to get property of non-object in .../VersionControl/ProcessVersionControl.module:654 #6

Closed
adrianbj opened this Issue Oct 12, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@adrianbj

adrianbj commented Oct 12, 2016

This happens when you have RepeaterMatrix fields (I don't think it affects normal Repeater fields) on the page.

The fields end up being named things like: body_repeater2143

Maybe not the best fix, but as I quick test, I confirmed that if you add this line before Line 654, it fixes it:

$field = preg_replace('/\d/', '', str_replace('_repeater','', $field));
@teppokoivula

This comment has been minimized.

Show comment
Hide comment
@teppokoivula

teppokoivula Feb 14, 2017

Owner

@adrianbj: sorry for taking so long to answer!

I've been trying to reproduce this issue, but so far no luck. It could be that something has changed, or perhaps I should say fixed, in the meantime – but just to make sure, you wouldn't happen to still have a test case available for this? :)

On a slightly related note, Repeaters don't seem to work too well with this module. I'm pretty sure they never did, so it's probably nothing new, more likely something to be added to the backlog.

Owner

teppokoivula commented Feb 14, 2017

@adrianbj: sorry for taking so long to answer!

I've been trying to reproduce this issue, but so far no luck. It could be that something has changed, or perhaps I should say fixed, in the meantime – but just to make sure, you wouldn't happen to still have a test case available for this? :)

On a slightly related note, Repeaters don't seem to work too well with this module. I'm pretty sure they never did, so it's probably nothing new, more likely something to be added to the backlog.

@teppokoivula

This comment has been minimized.

Show comment
Hide comment
@teppokoivula

teppokoivula Dec 6, 2017

Owner

No luck at reproducing this one, so I'm closing the issue.

Owner

teppokoivula commented Dec 6, 2017

No luck at reproducing this one, so I'm closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment