As described in title. there is an extra ')' at the end of line 25. Please remove it, thank you.

CommentFileSizeAuthor
#6 author-pane-824620.patch790 byteslarskleiner

Comments

michelle’s picture

Priority: Critical » Normal

Thanks for letting me know. Haven't tried testing with core profile, yet.

Michelle

Takafumi’s picture

Well, this typo is critical issue because update.php cannot be run.
Probably, it is good to correct early.

michelle’s picture

I will fix it next time I work on AP. It's not critical because it's only in the dev version that has a big warning that people should not be using anyway and only if you use it with core profile.

Michelle

Junro’s picture

Hello,

Confirm update.php can't run.

Sure this is not critical but we can't use this dev version, anyway we can't see the differencies between both version so, using this version or keep the old one :)

michelle’s picture

The dev is in the middle of huge changes that I haven't had a chance to finish, which is why there is big bold warnings not to use it. If you want to help test, great, just disable the profile module and you can test the rest. There's no need to run update.php as there is no database changes.

Michelle

larskleiner’s picture

Status: Active » Needs review
StatusFileSize
new790 bytes

This patch should fix it for now.

michelle’s picture

Thanks, larskleiner. I'm in the middle of rebuilding my dev site so haven't been able to get back to coding, yet.

Michelle

finex’s picture

I can confirm that the patch fix it :-)

michelle’s picture

LOL! Yeah, it's a pretty simple typo. I just don't have a development environment set up to commit anything right now and won't until early next week as I have a big photography project I need to get done by Monday.

Michelle

michelle’s picture

Status: Needs review » Fixed

Committed, thanks.

Michelle

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.