The only feature that worked for me when I installed this module was the label change on the submit button, but it changed the node save button and not the comment button. Settings changes for the other features of this module had no effect at all.

CommentFileSizeAuthor
#32 multiple_profiles.patch392 bytesmattiasj
#14 nodeformsettings.jpg133.42 KBlelizondo

Comments

lelizondo’s picture

can you please check in the database if the variables are being correcly saved? you can do it by opening phpmyadmin or something like that and running the following query:

SELECT * FROM variable;

If your tables have a prefix then you should add it before the name 'variable'.

you should see something like:

nodeformsettings_submit_nodetype
commentformsettings_size_nodetype

Example: I have a 'podcast' node type, I have the following:

nodeformsettings_submit_podcast
commentformsettings_size_podcast

Please post if you have something on the variable table, if not that could be a problem, somehow variables are not being saved.

jimsmith’s picture

I found the following in the variable table:

commentformsettings_author_blog		s:1:"0";
commentformsettings_preview_blog         s:1:"0";
commentformsettings_title_blog		s:1:"0";
commentformsettings_inputformat_blog	s:1:"0";
commentformsettings_size_blog		s:2:"15";
commentformsettings_submit_blog		s:4:"Save";
nodeformsettings_splitsummary_blog	s:1:"0";
nodeformsettings_inputformat_blog	        s:1:"1";
nodeformsettings_preview_blog		s:1:"1";
nodeformsettings_submit_blog		s:4:"Save";
lelizondo’s picture

Are you using the default blog content type drupal provides via the blog module? What theme are you using? I'll try to replicate the error since this is weird.

jimsmith’s picture

Yes, it is the default blog content type. I also tried it with the default story content type and had the same results. I've not yet tried it on a custom content type.

Juryiel’s picture

Same here, it's not working. Using Deco theme and it doesn't seem to work on any content type i've tried.

lelizondo’s picture

Priority: Normal » Critical

I'll look into it tomorrow mourning (in a few hours). I'm not having any issues in the only site I'm using this module but probably I'm just using a version I haven't commited.

Juryiel’s picture

If you need any more info let me know and I can try to provide it but I'm not sure what type of info would be useful.

lelizondo’s picture

It seems like there's problems with some sites, I've tried to create a new site with only this module and vertical_tabs but everything works as it should, I think there could be some conflict with another module. Can anybody please install a fresh site with only this module and vertical_tabs (optional) and confirm the module it's just broke?

It should work without a problem, if it does work as it should, please post the modules enabled on your non-working site so I can compare the list with one of my sites.

Thanks

jimsmith’s picture

Title: Comments form settings doesn't work when using Content Profile and Advanced Profile Kit » Doesn't work
Version: 6.x-2.x-dev » 6.x-1.x-dev
Category: support » bug
Priority: Normal » Critical
Status: Postponed (maintainer needs more info) » Active

I figured out what the problem was, at least for me: The much-dreaded operator error!

I didn't realize that many of the settings were within the comment settings section of the content type edit page. I was only using the settings under the Node form settings fieldset.

Maybe this points out a documentation issue.

I apologize for what seems to be an unnecessary bug hunt. However, I'll let you change the status to closed in case there are some conflicts or bugs I haven't seen.

Thank you, @lelizondob for the module. It's very helpful.

lelizondo’s picture

Category: bug » support
Priority: Critical » Normal

Good to know. I'll keep this issue open in case anyone has a problem. I should probably include a README.

One small thing, the only thing it's not working is when trying to disable the input form fieldset and the module better formats is enabled. I'll look into that later.

Juryiel’s picture

A few things:

I investigated further and it seems that it doesn't work on the comment form when I use advanced profile kit to display the Node Comment form in one of those pannels APK provides. However, if I visit the node it works. Can you look into this?

Second, the (Hide the word 'Comment'? ) option is misleading. When it is Enabled the word 'Comment' shows, when it is disabled the word 'Comment' is hidden. This is the reverse of what I would expect from the wording.

lelizondo’s picture

I'll check it out, never used the advanced profile kit but I'll give it a try

Juryiel’s picture

it might not have anything ot do with APK, but with panels. Try making a panel page and in one of the sections adding Comment Form (Under the Node category)

lelizondo’s picture

StatusFileSize
new133.42 KB

@Juryiel I did what you told me and created a panel page to rewrite the node display but I'm not having problems, the comment form it's being altered. Attaching a screenshot, maybe I misunderstood you.

lelizondo’s picture

Status: Active » Postponed (maintainer needs more info)
Juryiel’s picture

Yeah it might be just with APK. I've been busy moving to a new place but now that I'm done I'll run some tests on the new version to try and figure out where the problem is.

Juryiel’s picture

I"m kind of noob at drupal so let me add some more info since you might be better at investigating.

So APK creates (or you can import) a specific Panel Page, that then displays various parts of the User Profile content type it also creates. So if I edit the comment form settings of the User Profile content type, and I visit the User Profile nodes directly, comment form settings work fine. On the other hand, if I visit the Panel Page that displays it, it does not seem to take the settings into account. I tried creating a Panel Page separate from APK to display other nodes in, but i can't figure out how to do it and how to set it up properly, making it impossible for me to test outside of APK. But maybe you can, or if not, maybe use APK to do it also.

Juryiel’s picture

It does seem that the APK stuff is hard-coded and therefore might work differently than defualt panel pages. So maybe try testing with that directly?

lelizondo’s picture

I'll try with APK then.

Juryiel’s picture

Have you been able to replicate the problem? Or is it something on my end?

lelizondo’s picture

Status: Postponed (maintainer needs more info) » Active

Finally I replicated the problem, it's just APK, I been very busy but I'll try to fix the problem tonight.

Juryiel’s picture

Any progress on this issue? :) If you're having trouble, there is a module which does successfully alter those APK comment forms called 'Resizable Body' that might be helpful to look at.

lelizondo’s picture

Status: Active » Needs review

I've released a new dev version that fixes this bug. Please test

lelizondo’s picture

Title: Doesn't work » Comments form settings doesn't work when using Content Profile and Advanced Profile Kit
Juryiel’s picture

I just saw this and tested the new version. Works great, thanks!

lelizondo’s picture

Status: Needs review » Fixed
mattiasj’s picture

I tested this as well but can confirm that it does not work with Content profile when using multiple profiles. When doing this nodeformsettings chooses only the last of the profiles to alter the comment form for. I tested this using 5 content profile node types. The following change fixes it:

// User profile (When using content profile)
if(module_exists('content_profile')) {
	if(arg  == 'user' && is_numeric(arg(1))) {
	  $content = array();
	  foreach (content_profile_get_types('names') as $type => $type_name) {
	    $node = content_profile_load($type, arg(1));
	    if($node){ break; } // added this line
	  }
	}
}
lelizondo’s picture

can you please make a patch?

lelizondo’s picture

Status: Fixed » Active
mattiasj’s picture

I am not that experienced to make a patch (I am actually a themer), but I hope someone else can use my finding!

lelizondo’s picture

is actually very easy http://drupal.org/patch/create

I don't really know what lines you modified, that's why I need a patch..

mattiasj’s picture

StatusFileSize
new392 bytes

Here comes the patch.

lelizondo’s picture

Status: Active » Fixed

I'm sorry I didn't commit this earlier. I was really busy releasing a new module. I've just committed this to the latest dev. It should be ready in a few hours.

mattiasj’s picture

Great!

Status: Fixed » Closed (fixed)

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

fuerst’s picture

Status: Closed (fixed) » Active

The patch in #32 does not fix the problem: Now the first of multiple content profile nodes is choosen instead of the last one. I discovered this when using the user_view panel of the Panels module to display the contact form. In the case of Panels you need to know the context of the Panels-Content Profile-Relation to get the Content Type used in this relation. Don't have an idea currently how to implement this.

lelizondo’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Active » Postponed (maintainer needs more info)

This is a problem of the path, if the path changes and is not defined in the module, then the functionality will not be loaded. I can't add all possible paths but any standard/often-used path should be added.

In any case, this will be added to the 6.x-2.x branch. I'm postponing this until I hear back from you. Please move to active to call my attention. Thanks.

Status: Active » Postponed (maintainer needs more info)
kssundar’s picture

Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (won't fix)