Skip to content
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

Issue : body text not accepted. (news, featurebox menu/pages etc..) #1035

Closed
willem010 opened this issue May 31, 2015 · 32 comments
Closed

Issue : body text not accepted. (news, featurebox menu/pages etc..) #1035

willem010 opened this issue May 31, 2015 · 32 comments
Labels
status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing.

Comments

@willem010
Copy link

after creating or editing a newsitem, the item has no content.
body and extended text are not saved (when tinymce is used)

after editing :
System Information
Nothing updated as no changes were made.

i found:
when both body and extended text are set to bbcode,
it does save, but only when there are no images in the content.

@CaMer0n
Copy link
Member

CaMer0n commented Jun 8, 2015

Do you think it is related to server restrictions?

@willem010
Copy link
Author

might be.

but everything was running ok before updating to this latest git.
i think the working version was an old one tho, probablly november last yr,

@CaMer0n
Copy link
Member

CaMer0n commented Jun 8, 2015

News was updated to use the AdminUI. Pages, Welcome Message, FAQs - they all use the same mechanism, so I'm not sure what is causing the problem. I am unable to reproduce this issue with the news.

@willem010
Copy link
Author

it does seem to be a tiny issue,
i just tried changing a featurebox item, same issue..
for some reason it's not accepting text, which IS posted.
i do get a 302 response tho..
followed by a 200. dunno if thats supposed to happen here, but the second response seems ok.
and all other fields are accepted.. so .. i dont get it ,, haha

@willem010
Copy link
Author

this seems interesting :
it should have been posting "test test test" as text entered in tiny

edit - which IS posted .. so it's not interesting ..
removed the content...

@willem010
Copy link
Author

i have been looking at this one again,
the issue is with the gabber.fm site, which really needs frequent news updates.

tried uninstalling tiny.. but that didnt help at all,- problem is not with tiny
(so topic should be renamed. :P )

then i noticed the same issue, empty body when adding a new menu.

due to some issues in the team on this site, i am not gonna get the admin logs. people dont want to give me the acces i need for this,

please help me on this one.
i really have no idea where to go from here.

@willem010 willem010 changed the title Issue : TINYmce edit / posting results in empty newsitem. Issue : body text not accepted. (news, featurebox menu/pages etc..) Jun 27, 2015
@willem010
Copy link
Author

bughunt session #4

posting plain text -> success :) .. "test test" .. no problem.
posting text with tags -> blank result. ( 'test test <' doesn't work)

so i tried html specific chars ..
the problem is with the use of <, > or &
as soon as the body contains one of those, the post will be blank.

i tried
renaming my htaccess (to check if there is a problem with it )
disable wysiwyg, profanity filter, clickable urls etc.
tiny uninstalled,
checking posting credentials, allow html posting & abuse filter ..
debug mode on (but the posting isnt shown anyway, just the results of the page reloading are shown.. )

nothing helps, but i now suspect it's an regexp somewhere ??

btw,git version of last night.,

@willem010
Copy link
Author

e107_err

can anyone tell me where to start my search?

@CaMer0n
Copy link
Member

CaMer0n commented Jun 28, 2015

@willem010 Have you tried reproducing this on different servers?

@willem010
Copy link
Author

yeah i tried. with no success..
pretty similar installs, about the same version of everything.. and those run fine.
but most of those are v2.0 installs,
this buggy one still is the ancient upgraded v1 site...

can you at least point me in a direction to start hunting for whats causing this?
it was ok on the git from november 2014.
what part is handling the posted form data?

@CaMer0n
Copy link
Member

CaMer0n commented Jun 28, 2015

disable htaccess, and add this after class2.php in newspost.php (but only once you're ready to submit the form)

print_a($_POST);
exit; 

Also, try exporting preferences from a fresh install, and importing into the problem site. (backup everything first of course) If you don't have access to post HTML, it will be stripped.

@willem010
Copy link
Author

$_post gives me all the posted data,
news_body just as it was posted, [html]

test

[/html]
looks ok up till here.

but a few lines down, it shows
System Information
Nothing updated as no changes were made.

and the body field is empty.

@CaMer0n
Copy link
Member

CaMer0n commented Jun 28, 2015

okay, sounds like a permissions issue with html posting.

@willem010
Copy link
Author

if you say so.. haha..

i would think the data is received by newspost, which just should post it to the database... ?
posting fields like this used to be no problem in the past.. so why is it now ?

anyway.. it's a dedicated server,
if we need to update something to make it work again, maybe we should hire our linux guru to fix it?
any suggestions?

@CaMer0n
Copy link
Member

CaMer0n commented Jun 29, 2015

If you're seeing what is posted using print_a (complete with HTML), then it is has nothing to do with Linux or a server config.

@CaMer0n
Copy link
Member

CaMer0n commented Jun 29, 2015

Please, can you go to Admin -> Database -> Preferences Editor. And tell me the value for 'post_html' ?

@willem010
Copy link
Author

ah ok. i misunderstood..

the value is 254..

admins should be able to post html.
my account = main admin.

@willem010
Copy link
Author

i did notice a real mess in my usertable and userclasses.

i tried to clean up the classes using a copy from a fresh install,
now at least the tree makes some sense
(in my copy searchengines was root ? and classes showed no parents.. thats fixed now)

but .. still not able to post html,

doesnt matter what user level i set for posting html in the prefs,
fiddling with the tree doesnt help. adding more memberships to my account neither.
even plain and simple -> everyone allowed to post html .. still same issue.

@Moc Moc added the status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. label Jul 5, 2015
@willem010
Copy link
Author

updated to current git. still
"System Information
Nothing updated as no changes were made."

is there anything i can do to help fixing this?

@CaMer0n
Copy link
Member

CaMer0n commented Jul 27, 2015

@willem010 I started to notice this issue too recently. Pages and FAQs has no problem for me. But News does. . You can check the 'body' field data by enabling SQL debug.

ie. newspost.php?[debug=showsql+]

@willem010
Copy link
Author

this shows me the body text is empty.

at top of the newpost.php i still have the print_a($_POST); which shows me the body text.
so the data gets lost somewhere down the line.

@willem010
Copy link
Author

1
2
3

all from the same page.

CaMer0n added a commit that referenced this issue Jul 27, 2015
@CaMer0n
Copy link
Member

CaMer0n commented Jul 27, 2015

@willem010 Could you please login at e107.org and then use the contact form to send me your skype address? We need to get the bottom of this!

@willem010
Copy link
Author

I dont use skype, but i will contact you anyway.
(Y)

CaMer0n added a commit that referenced this issue Jul 28, 2015
…hanges are precautions. Still not sure what is causing it.
@Moc
Copy link
Member

Moc commented Oct 24, 2015

@willem010 Is this issue still present using the latest files?

@willem010
Copy link
Author

still present with pretty recent files but i will check latest this week.

@Moc
Copy link
Member

Moc commented Oct 24, 2015

That's fine, just wanted to make sure since the last message about this was a few months ago. Thanks.

@willem010
Copy link
Author

yeah.. been busy. didnt take much time to look at this again. i did try files from about a month back. didnt solve it.

@CaMer0n
Copy link
Member

CaMer0n commented Nov 5, 2015

I believe this was a TinyMce issue, which has since been fixed. But if you can test and confirm that would be appreciated.

@willem010
Copy link
Author

i will check first thing tomorrow.

@willem010
Copy link
Author

updated files.
( i will report a few new bugs in new topic. )
but for this one :

problem still exists.

site reports no update to text.
err2

@willem010
Copy link
Author

closing this one, as this most probably was caused by a hacked server.

im unable to reproduce on the newly installed server.

we cannot find what was causing this (i really have no idea what happened )
but its more likely to be a server (config) issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing.
Projects
None yet
Development

No branches or pull requests

3 participants