e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
322 stars 214 forks source link

Cannot modify users information #590

Closed dimante closed 10 years ago

dimante commented 10 years ago

A couple strange things with this one: image

The good old LAN_??? which means some language file mapping is not present?

When trying to update a persons email address etc the following occurs:

image

Almost like it is trying to do an insert instead of database update.

Thanks -D-

Moc commented 10 years ago

I don't quite understand why you are getting that strange layout... are you completely sure all the files are uploaded correctly on your installation? That sidebar on the right should not be there at all , and the layouts of the fields is not correct either.

dimante commented 10 years ago

​you mean the right side layout? It is a setting.... I have an unmodified bootstrap 3 theme... Not sure what you mean...​

On Thu, May 22, 2014 at 4:56 PM, Tijn Kuyper notifications@github.comwrote:

I don't quite understand why you are getting that strange layout... are you completely sure all the files are uploaded correctly on your installation? That sidebar on the right should not be there at all , and the layouts of the fields is not correct either.

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-43949133 .

Moc commented 10 years ago

The site info bit... It's not supposed to be there at all. Here's how it looks on my installaton:

image

Sidenote: because of how I cropped the image the user and home icon dropdown are missing on top right hand side.

dimante commented 10 years ago

I am not doing any customization on the theme.... So short answer is I am not sure.... This was an upgraded v1 website. COuld that be part of the issue?

On Thu, May 22, 2014 at 5:01 PM, Tijn Kuyper notifications@github.comwrote:

The site info bit... It's not supposed to be there at all. Here's how it looks on my installaton:

[image: image]https://cloud.githubusercontent.com/assets/577978/3060864/92d5aeb6-e1fc-11e3-87c6-ed2133e4142f.png

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-43949597 .

Moc commented 10 years ago

It may not be limited to just the theme. Please do a full reupload of all the files from a freshly downloaded ZIP from Github. See my note in issue #533

dimante commented 10 years ago

Ok brother there is no way I have old php or other files... Here's what I did

Deleted EVERY file from my website except for e107_config. Went and got the latest github and downloaded it. I am telling you I have modified nothing and left the files exactly like they were in the archive... Cleared my Google Chrome Cache, Cleared the e107 cache... Went to the same place for the admin/user page and here's what I see:

[image: Inline image 1]

So with all caches cleared what would explain this now?

On Thu, May 22, 2014 at 5:05 PM, Tijn Kuyper notifications@github.comwrote:

It may not be limited to just the theme. Please do a full reupload of all the files from a freshly downloaded ZIP from Github. See my note in issue

533 https://github.com/e107inc/e107/issues/533

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-43950024 .

Moc commented 10 years ago

Unfortunately I cannot see the image. I take it the layout has remained the same? Edit: I can actually see it myself as well as you gave me admin access. Checking it out.

dimante commented 10 years ago

Yes I even looked at it in a browser I have never viewed it in before Safari... Looks the same.

[image: Inline image 1]

On Thu, May 22, 2014 at 5:28 PM, Tijn Kuyper notifications@github.comwrote:

Unfortunately I cannot see the image. I take it the layout has remained the same?

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-43951929 .

dimante commented 10 years ago

THis is pure unedited GITHUB files... So is the issue the v1 to v2 database? Seems like the only thing left...

On Thu, May 22, 2014 at 5:30 PM, John Gates dimante@dimante.net wrote:

Yes I even looked at it in a browser I have never viewed it in before Safari... Looks the same.

[image: Inline image 1]

On Thu, May 22, 2014 at 5:28 PM, Tijn Kuyper notifications@github.comwrote:

Unfortunately I cannot see the image. I take it the layout has remained the same?

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-43951929 .

tgtje commented 10 years ago

Was your update from v1 to v2 successfull ? If not : dbase could indeed be of influence.. (theme in use, completion of updating) Updating v1 to v2 = still troublesome

dimante commented 10 years ago

All indications were that it was.

On Fri, May 23, 2014 at 1:45 PM, leo notifications@github.com wrote:

Was your update from v1 to v2 successfull ? If not : dbase could indeed be of influence.. (theme in use, completion of updating)

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-44046730 .

dimante commented 10 years ago

Here's another strange finding... If the querystring is not used on user settings the page renders like this: screen shot 2014-05-24 at 8 25 13 am

And when the querystring is used:

screen shot 2014-05-24 at 8 24 19 am

THis to me is really peculiar.... I have gone through every table in the MySQL database... I have completely removed all web content except for e107_config and replaced it with GIT content... I have only 1 conclusion.. There is a problem with the bootstrap3 theme...

Let me know your thoughts on this... I am certainly unsure why in any form I am unable to update and save user information or why even though real name is not indicated as a required field it is requiring a value..

-D-

dimante commented 10 years ago

More info on errors happening:

[19-May-2014 20:23:34 America/Chicago] PHP Fatal error: Call to undefined method e_parse::toGlyph() in .\e107_themes\bootstrap3\theme.php on line 21 [19-May-2014 20:23:52 America/Chicago] PHP Fatal error: Call to undefined method e_parse::toGlyph() in ..\e107_themes\bootstrap3\theme.php on line 21 [20-May-2014 01:44:36 GMT] PHP Fatal error: Call to a member function tablerender() on a non-object in ..\e107_plugins\newsletter\admin_config.php on line 519 [20-May-2014 01:44:51 GMT] PHP Fatal error: Call to a member function tablerender() on a non-object in ..e107_plugins\newsletter\admin_config.php on line 519

PHP Version 5.5.11

-D-

Moc commented 10 years ago

Fixed the last two errors. The first two were already fixed in https://github.com/e107inc/e107/commit/2780e3af30111fb990c224019473d60baa7e3dba

dimante commented 10 years ago

Thanks!

On Sat, May 24, 2014 at 5:44 PM, Tijn Kuyper notifications@github.comwrote:

Fixed the last two errors. The first two were already fixed in 2780e3ahttps://github.com/e107inc/e107/commit/2780e3af30111fb990c224019473d60baa7e3dba

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-44102046 .

dimante commented 10 years ago

Latest GIT exact same problem...

dimante commented 10 years ago

Here is what happens: image

And after attempting to change the email:

image

Notice I still see the LAN_UPDATE and the requirement for the real name which again does not show required...

image

Since I have pure GIT software with no mods this should not be happening..

image

Moc commented 10 years ago

This is an upgrade from v1 right?

Just to confirm, please setup a separate new installation using the latest Github files and let us know if you are experiencing the same issue on a fresh installation as well.

dimante commented 10 years ago

Here is my question. I am cool with doing this but here is what I hoping you can give me...... Can you provide me a copy of a pristine e107 V2 database? Let me use my skills and do a diff and see what is different from the database I have now and a pristine e107 database.. I would like it to come from you so you can certify that the database you give me is a clean v2 install. I will diff it and figure out what is causing the issue.

On Sat, May 24, 2014 at 6:34 PM, Tijn Kuyper notifications@github.comwrote:

This is an upgrade from v1 right?

Just to confirm, please setup a separate new installation using the latest Github files and let us know if you are experiencing the same issue on a fresh installation as well.

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-44104982 .

Moc commented 10 years ago

It would be far more effective and efficient do a quick test install on your environment (windows & IIS) as I do not have such testing environment available (only Windows/apache and Linux/apache). Also, it will first verify that there is indeed an issue when upgrading from v1 to v2, so after confirming this, I'll be glad to provide you with a clean database install which you can then compare. I first want to rule out the environmental influences.

dimante commented 10 years ago

Ok.. I can do this later today. It should not take to long to set it up.

On Sun, May 25, 2014 at 3:19 AM, Tijn Kuyper notifications@github.comwrote:

It would be far more effective and efficient do a quick test install on your environment (windows & IIS) as I do not have such testing environment available (only Windows/apache and Linux/apache). Also, it will first verify that there is indeed an issue when upgrading from v1 to v2, so after confirming this, I'll be glad to provide you with a clean database install which you can then compare. I first want to rule out the environmental influences.

— Reply to this email directly or view it on GitHubhttps://github.com/e107inc/e107/issues/590#issuecomment-44124384 .

dimante commented 10 years ago

Here is some bad news. Fresh e107 install, fresh database, same issue on the screen when coming to the age the first time:

Object Moved

This document may be found hereHTTP/1.1 302 Moved Temporarily Cache-Control: private, max-age=10800, pre-check=10800 Content-Type: text/html; charset=UTF-8 Expires: Thu, 19 Nov 1981 08:52:00 GMT Last-Modified: Sat, 24 May 2014 20:58:56 GMT Location: index.php Server: Microsoft-IIS/8.5 X-Powered-By: ASP.NET Date: Sun, 25 May 2014 19:49:58 GMT Content-Length: 132 Object Moved

I am testing updating a user and I will also test the initial user group assignments.. More to come.

dimante commented 10 years ago

So with the fresh install I was able to update the user.. However, even though real name is not set to required or marked with an asterisk it still would not let me update the user without something in the real name field... So what is failing in the 1.0 to 2 update that my main site does not work....

-D-

Moc commented 10 years ago

Closing this issue for now as it's not reproduced thus far. If you need any info or if you have new information to share, feel free to re-open.

Moc commented 10 years ago

SQL Analysis 1 Query Errors! Index Query / Error 24 SELECT SQL_CALC_FOUNDROWS e107.* FROM e107_ LIMIT 0, 20 PHP Errors: Warning: in_array() expects parameter 2 to be array, string given, Line 499 of e107_handlers\user_handler.php

I've reproduced this. Will fix.

dimante commented 10 years ago

I appreciate that but it still does not explain why the page is rendering like that, right? THe steps I am using to get to that screen looking like that is. Admin > Users then choose edit from the dropdown list for any user and that screen is what renders. This is a bug somehow with the bootstrap admin theme.. It has to be I just don't understand why the button comes up LAN_UPDATE something in v2 is causing this I looked over my database in detail and I really don't see how it would be the problem. As I stated earlier in the case. The flat PHP files have all been replaced with pure GIThub E107 files. I hope we can get a chuckle when what's causing this is figured out :-)

On Thu, Aug 14, 2014 at 5:17 AM, Tijn Kuyper notifications@github.com wrote:

SQL Analysis 1 Query Errors! Index Query / Error 24 SELECT SQL_CALC_FOUNDROWS e107.* FROM e107_ LIMIT 0, 20 PHP Errors: Warning: in_array() expects parameter 2 to be array, string given, Line 499 of C:\inetpub\wwwroot\DCSLLCMAINWEB\e107_handlers\user_handler.php

I've reproduced this. Will fix.

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52166719.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

Moc commented 10 years ago

I'll do a test update tomorrow and see if I can reproduce the issue then.

dimante commented 10 years ago

Thanks!

On Thu, Aug 14, 2014 at 4:56 PM, Tijn Kuyper notifications@github.com wrote:

I'll do a test update tomorrow and see if I can reproduce the issue then.

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52250487.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

Moc commented 10 years ago

Very sorry but I cannot reproduce the issue on an active v1 to v2 update either. I think I remember an issue where the 'edit user' page would show up broken but I think it's different to yours. Will try and see if I can find it again. Edit: It's this one: https://github.com/e107inc/e107/issues/364 but I don't think it's relevant.

The user edit information does indeed display in the admin area (which I think is done correctly) but still it is broken in your screenshot. It's properly rendered in my screenshot (similar to https://github.com/e107inc/e107/issues/590#issuecomment-43949597)

dimante commented 10 years ago

DOes not make sense... The PHP flat files are all from GITHub source. When I rename the file I mentioned in earlier correspondence that weird extra info disappears... That file holds to secret to what is going on.

On Fri, Aug 15, 2014 at 11:12 AM, Tijn Kuyper notifications@github.com wrote:

Very sorry but I cannot reproduce the issue on an active v1 to v2 update either. I think I remember an issue where the 'edit user' page would show up broken but I think it's different to yours. Will try and see if I can find it again.

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52325620.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

Moc commented 10 years ago

No I disagree. That file is not the culprit here. Please note even after changing that file, you still have the wrong layout. There should be no 'Siteinfo' bar there... Please refer to my screenshot posted above.

@SecretR Do you have any clue what this could be?

dimante commented 10 years ago

That file that causes the weird output on the left and right sides is:

e107_core\shortcodes\batch\admin_shortcodes.php

Also, the action button coming up LAN_UPDATE instead of text seems to me that something is not linking up. I am running this on WIndows and IIS.. I am not sure how that could contribute to the strange display but figured I would add the info..

On Fri, Aug 15, 2014 at 11:20 AM, John Gates dimante@dimante.net wrote:

DOes not make sense... The PHP flat files are all from GITHub source. When I rename the file I mentioned in earlier correspondence that weird extra info disappears... That file holds to secret to what is going on.

On Fri, Aug 15, 2014 at 11:12 AM, Tijn Kuyper notifications@github.com wrote:

Very sorry but I cannot reproduce the issue on an active v1 to v2 update either. I think I remember an issue where the 'edit user' page would show up broken but I think it's different to yours. Will try and see if I can find it again.

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52325620.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

Moc commented 10 years ago

Do you have access to a linux environment at all? If yes, it would be interesting to perform the upgrade there as well. See if it makes any difference at all. I'm out of ideas again. I've also notified Miro (SecretR) so when he has some time, he'll help out where possible.

LAN_UPDATE means it has not included the language file (which would be the generic ADMIN English file, e107_languages/English/admin/lan_admin.php (included on every admin page). Your 'Edit user' screen is rendered in a FRONTEND setting, not admin. Therefore it does not include the LAN.

dimante commented 10 years ago

to the thorough here is my e107config file (passwords removed)

<?php /*

$mySQLserver = ''; $mySQLuser = ''; $mySQLpassword = ''; $mySQLdefaultdb = 'DCSE107'; $mySQLprefix = 'e107_'; /define('e_SECURITY_LEVEL', 5); / define('e_POWEREDBY_DISABLE', 'true');

$ADMIN_DIRECTORY = 'e107_admin/'; $FILES_DIRECTORY = 'e107_files/'; $IMAGES_DIRECTORY = 'e107_images/'; $THEMES_DIRECTORY = 'e107_themes/'; $PLUGINS_DIRECTORY = 'e107_plugins/'; $HANDLERS_DIRECTORY = 'e107_handlers/'; $LANGUAGES_DIRECTORY = 'e107_languages/'; $HELP_DIRECTORY = 'e107_docs/help/'; $MEDIA_DIRECTORY = 'e107_media/'; $SYSTEM_DIRECTORY = 'e107_system/';

On Fri, Aug 15, 2014 at 11:24 AM, Tijn Kuyper notifications@github.com wrote:

No I disagree. That file is not the culprit here. Please note even after changing that file, you still have the wrong layout. There should be no 'Siteinfo' bar there... Please refer to my screenshot posted above.

@SecretR https://github.com/SecretR Do you have any clue what this could be?

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52327125.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

dimante commented 10 years ago

Ok. If I send you my database I would be curious if your site then displays like mine... I kind of feel like if this is not flat file causing this maybe the database is... I compared a fresh v2 to my upgrade though and I did not see a difference. It is certainly maddening though.

On Fri, Aug 15, 2014 at 11:26 AM, Tijn Kuyper notifications@github.com wrote:

Do you have access to a linux environment at all? If yes, it would be interesting to perform the upgrade there as well. See if it makes any difference at all. I'm out of ideas again. I've also notified Miro (SecretR) so when he has some time, he'll help out where possible.

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52327351.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

myovchev commented 10 years ago

For some reason Real Name is required field. Different Display/Login name is also there due to Display name restriction. Go through your site settings - Signup, Security & protection sections. Different layouts - user/edit route is Front-end one, user/edit/XXX is admin area.

Moc commented 10 years ago

@SecretR But why does @dimante's install show up broken then? (the site info bar on the right for example) whereas my installation shows up in the proper layout?

Moc commented 10 years ago

If I send you my database

You can mail it to me on e107@tijnkuyper.nl :)

dimante commented 10 years ago

So the database helped with the issue? I am going to apply this and see how it does :-)

On Wed, Aug 20, 2014 at 7:02 AM, Cameron notifications@github.com wrote:

Closed #590 https://github.com/e107inc/e107/issues/590 via 082f26c https://github.com/e107inc/e107/commit/082f26c2b9a304f483cb80ad8bf249c367b1a7d6 .

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#event-154958156.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

Moc commented 10 years ago

@dimante No, I haven't even looked at your database yet or forwarded it to Cameron whatsoever :)

Apparently @CaMer0n managed to reproduced the issue on his installations.. ?

dimante commented 10 years ago

Yeah, well that is refreshing.. I was starting to think I was loosing my mind =-)

On Wed, Aug 20, 2014 at 7:13 AM, Tijn Kuyper notifications@github.com wrote:

@dimante https://github.com/dimante No, I haven't even looked at your database yet or forwarded it to Cameron whatsoever :)

Apparently @CaMer0n https://github.com/CaMer0n managed to reproduced this on his installations.. ?

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52766802.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

Moc commented 10 years ago

Haha, well you didn't :) Is it all working properly now?

dimante commented 10 years ago

It is all working and displaying correctly:

[image: Inline image 2] Awesome! I am still confused as to how all people's sites were not looking like this. What was the root cause?

On Wed, Aug 20, 2014 at 12:04 PM, Tijn Kuyper notifications@github.com wrote:

Haha, well you didn't :) Is it all working properly now?

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52808769.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

dimante commented 10 years ago

And good work to all who worked on this!

On Wed, Aug 20, 2014 at 4:34 PM, John Gates dimante@dimante.net wrote:

It is all working and displaying correctly:

[image: Inline image 2] Awesome! I am still confused as to how all people's sites were not looking like this. What was the root cause?

On Wed, Aug 20, 2014 at 12:04 PM, Tijn Kuyper notifications@github.com wrote:

Haha, well you didn't :) Is it all working properly now?

— Reply to this email directly or view it on GitHub https://github.com/e107inc/e107/issues/590#issuecomment-52808769.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

This email may contain information that is confidential or attorney-client privileged and may constitute inside information. The contents of this email are intended only for the recipient(s) listed above. If you are not the intended recipient, you are directed not to read, disclose, distribute or otherwise use this transmission. If you have received this email in error, please notify the sender immediately and delete the transmission. Delivery of this message is not intended to waive any applicable privileges.

Moc commented 10 years ago

I don't know what caused the differences between your and my installation, that's one of those annoying weird things when bugfixing. It's one thing for it to be a bug, but sometimes they're real annoying and they behave differently on each occurrence, making them really hard to squash...

Anyhow, the entire admin area is in the process of being converted over to the new Admin UI (user interface). This is an ongoing process which takes a while. In the mean time, some areas are converted and some are not. The ones that aren't are still functional on legacy code but they may be giving issues (such as the one you experienced). Cameron fixed your issue by converting the 'edit user' part to the new Admin UI. Hope this explains some bit at least :)