Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amysteen’s picture

I am looking for this functionality as well.

For my site it's really just that I want multiple images for one category, but not others.

amysteen’s picture

I found just using cck, categories and node expire was a better route for this particular project that Im working on... (I am still using this module on another site though, so dont feel bad hehe)

fgm’s picture

Status: Active » Fixed

@amysteen found a workaround.

This will not be provided in 5.x or 5.x anyway. Maybe 7.x or 8.x ...

joekrukosky’s picture

Version: 5.x-1.5-8 » 7.x-3.0-alpha1
Category: support » feature
Status: Fixed » Active

I'd like to re-open this as a possible 7.x-3 feature, with the understanding it may need to be for the next major release, e.g. 7.x-4. This could allow the basic settings and notifications to be per content type. It would also solve the issue at: http://drupal.org/node/1209362.

fgm’s picture

Status: Active » Postponed

As you surmised, not for 7.x-3.0: this is essentially 6.x-3.x for Drupal 7. Could happen later, so marking postponed: postponed issue will be revisited after 7.x-3.0.

fgm’s picture

Status: Postponed » Closed (duplicate)

#350938: Support multiple node types in Classified Ads is a duplicate of this, but actually has code supporting the idea.

firstlevel’s picture

Version: 7.x-3.0-alpha1 » 7.x-3.0
Status: Closed (duplicate) » Active

would be very happy, if these feature could be implement. if we can get in with you, to support these feature may as sponsoring, pls get in contact with us. looking very forward for that

fgm’s picture

Version: 7.x-3.0 » 7.x-3.x-dev

I'm afraid we have no development time available before february to tackle this (D8 core work + existing customers): the best for you if you are interested in that feature is probably like this:

  • You confirm you are willing to sponsor the work
  • I post a note about this on the project page
  • You post a request to the Drupal consultants mailing list: that should probably get you direct contact with other devs.
  • If a dev creates that features with the normal quality process (hint: Test first, code later), I will merge it to the module and create a 7.3.1 to include it with the other changes committed since 7.3.0.
  • Of course, you can also contact suppliers on your local market, but this is likely to be more expensive (but faster)
firstlevel’s picture

reply via pm/contact

--> open for customers which will support the idea or can help in programming. pls get in contact here or via pm. thx

firstlevel’s picture

did not get any reply of my email to you... :(
so it would be not working to get it in the 7.3.1

fgm’s picture

Sorry, I seem to remember a private exchange at that time indeed and probably a lack of time to answer: if you are still looking for this, please contact me directly again: I may have something for you.

firstlevel’s picture

doing tomorrow, thanks for your reply. (just wanne try to support a bit...if we can)

spiritfelix’s picture

I am working on this also, maybe i can give a patch for this days later...

spiritfelix’s picture

Please Help Me to Test This, Thanks!

spiritfelix’s picture

Please Help Me to Test This, Thanks!

fgm’s picture

Status: Active » Needs review

Setting to needs review for bot to check the patch.

fgm’s picture

For reference, there was an patch suggested at http://drupal.org/node/350938#comment-7290360 for a much earlier version.

Status: Needs review » Needs work

The last submitted patch, use_different_content_type-195494-comment-7289976.patch, failed testing.

fgm’s picture

Error is located in classified_install: the query is incorrectly formed.

Exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '))' at line 1' in /var/lib/drupaltestbot/sites/default/files/checkout/includes/database/database.inc:2168
Stack trace:
#0 /var/lib/drupaltestbot/sites/default/files/checkout/includes/database/database.inc(2168): PDOStatement->execute(Array)
#1 /var/lib/drupaltestbot/sites/default/files/checkout/includes/database/database.inc(680): DatabaseStatementBase->execute(Array, Array)
#2 /var/lib/drupaltestbot/sites/default/files/checkout/includes/database/select.inc(1264): DatabaseConnection->query('SELECT nt.type ...', Array, Array)
#3 /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/ed_classified/classified.module(1344): SelectQuery->execute()
#4 [internal function]: classified_node_info()
#5 /var/lib/drupaltestbot/sites/default/files/checkout/includes/module.inc(864): call_user_func_array('classified_node...', Array)
#6 /var/lib/drupaltestbot/sites/default/files/checkout/modules/node/node.module(722): module_invoke('classified', 'node_info')
#7 /var/lib/drupaltestbot/sites/default/files/checkout/modules/node/node.module(476): _node_types_build(true)
#8 /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/ed_classified/classified.install(80): node_types_rebuild()
#9 [internal function]: classified_install()
#10 /var/lib/drupaltestbot/sites/default/files/checkout/includes/module.inc(864): call_user_func_array('classified_inst...', Array)
#11 /var/lib/drupaltestbot/sites/default/files/checkout/includes/module.inc(477): module_invoke('classified', 'install')
#12 /var/lib/drupaltestbot/sites/default/files/checkout/modules/simpletest/drupal_web_test_case.php(1476): module_enable(Array, true)
#13 /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/ed_classified/tests/classified_notifications.test(36): DrupalWebTestCase->setUp('classified', 'classified_noti...')
#14 /var/lib/drupaltestbot/sites/default/files/checkout/modules/simpletest/drupal_web_test_case.php(501): ClassifiedNotificationsTest->setUp()
spiritfelix’s picture

Status: Needs work » Active
FileSize
19.59 KB

can u test this new patch? thank u!
patch for 7.x-3.x

sorry, wrong file

spiritfelix’s picture

Status: Active » Needs review
FileSize
13.06 KB

plz review this one, thanks!

Status: Needs review » Needs work

The last submitted patch, use_different_content_type-195494-comment-7289976-2.patch, failed testing.

spiritfelix’s picture

Status: Needs work » Needs review
FileSize
13.01 KB

fix sth, check again, thx.

Status: Needs review » Needs work

The last submitted patch, use_different_content_type-195494-comment-7289976-3.patch, failed testing.

fgm’s picture

Seems like you're making progress, don't be afraid: initially, you had no passing test and now 296 ! No need to ask for a check, BTW: whenever you set the issue to needs review, the test bot will perform a new test pass.

spiritfelix’s picture

Yep, thank u, now I am working in this locally and run test in my local server now.
This is my first time to work with "test", so, maybe a little slow.
: )

System Lord’s picture

Issue summary: View changes

I'd also be willing to sponsor some if someone wants to work it.

SL

fgm’s picture

I started some work on this at the time, but it is definitely not finished. You can either

  • either apply the numbered patches in order, 0001 to 0006, committing each time, to get the progressive steps of work
  • or apply the combined unnumbered patch to get all changes at once
fgm’s picture

Status: Needs work » Closed (works as designed)

With this module end of life approaching along with Drupal 7 EOL, this issue is now moot.