The current development snapshot is to old to function in current d7 and we need to port this so users don't continue to use such an outdated 7.x-dev release.

Please port!!! kthx

Comments

cesareaugusto’s picture

subscribe :) when will a drupal 7 version be available?

KirstenLangholz’s picture

Waiting impatiently for a D7 version, too.

missym’s picture

Subscribe

Tilt_11’s picture

subscribe

nopulse’s picture

Port to 7 would also be much appreciated. Thanks.

likewhoa’s picture

Title: Port to Drupal 7 » Port Classified Ads to Drupal 7
Priority: Normal » Major
tesmon’s picture

subscribe

Cray Flatline’s picture

Subscribe. Any news?

david88’s picture

subscribe

fgm’s picture

No 7.x branch foreseen until we have a full 6.x-3.x version with 0 non-minor bug remaining, unless someone else want to try their hand at it.

likewhoa’s picture

Status: Active » Postponed

^ well that sucks...

causalloop’s picture

second that...

fgm’s picture

Well, if you want to try your hand at porting 6.3 to D7, I have no objection and it could be committed if it mostly works: I just don't have time to do it now. Want to try ?

fgm’s picture

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

For anyone willing to work on it, I have started the process: there is currently a 7.x-3.x branch that has working code similar to the 6.3 branch, plus a number of upgrade fixes to do.

Since the code is not (yet) for end-users, the dev release is not made visible on the project page.

Removing tag: it duplicates the module.

fgm’s picture

Status: Postponed » Needs work

Underway : all hands welcome to test, notify about errors, and suggest patches.

likewhoa’s picture

thanks fgm, this module is great.

fgm’s picture

Today's changes :

  1. installer now properly creates body and category fields
  2. uninstaller now properly removes fields, terms, vocabulary, nodes and variables
  3. classified.scheduled.inc ported to DBTNG: should be complete
  4. classified_notifications: installer for QueueAPI, removed dependency on job_queue (not ported to D7)
  5. all references to current time updated to REQUEST_TIME

If you want to help, please write test cases!

fgm’s picture

Changes in today's dev version:

  1. All 3 blocks ported to D7. DBTNG and better themability
  2. Advanced help now contains a page about theming blocks
  3. hook_taxonomy() ported: module now reacts properly to category changes in spite of caching
fgm’s picture

Changes in today's version

- list of Ads on user profil now works in all 3 modes: simple list, node teaser, node view in "classified" mode
- Ad pages now work
- expires extra-field can now be positioned per-view-mode in Field UI
- small CSS adjustments for expires block and simple list

likewhoa’s picture

just wanted to drop by and say a big thanks to you fgm for taking the time to do this port. +1 for you and irc karma if I ever see you on there.

fgm’s picture

Changes in today's version

  • overview page (/classified) now links to Classified Ads own taxonomy pages, not default ones,
  • overview page has an invisible (by default) class when rendered from cache
  • module taxonomy pages (/classified/(tid)) now in all 3 modes: simple list, node teaser, node view in "classified" mode, same as list on user profile
  • breacrumbs on taxonomy pages now honor drupal standard (do not include page title)
  • expires extra-field can now be positioned per-view-mode in Field UI
  • node type hooks adjusted
LagPRO-dupe’s picture

Thanks for porting this to Drupal!

I did manage to come across an issue. On the Taxonomy page under Operations, the "edit vocabulary", "list terms" and "add terms" seem to have the URL missing from the link:

  • edit vocabulary shows: admin/structure/taxonomy//edit
  • list terms shows: admin/structure/taxonomy/
  • add terms shows: admin/structure/taxonomy//add

I'm using the 2011-Aug-05 release with Drupal 7.0 (I'm upgrading right now to the latest 7.7)

fgm’s picture

Confirmed. Looking into it, thanks for reporting.

Btw changes in yesterday's version

  • new feature: a "Classified Link" formatter usable on any term reference field to link to the Classified Ads page for the term instead of the default term page
  • bug fix: fixed parameters error in _classified_taxonomy() → might be related to that problem
  • bug fix: total ad count on main Classified page only counted the top level categories
  • misc: updated node type hooks

For you guys looking at the D7 version, it would be better for tracking if you would open new issues on the 7.x branch for individual problems you encouter so they can be referenced in the commit messages when they are fixed: this one will be just a progress report.

fgm’s picture

There were actually two issues in this #1241214: Uninstall fails and #1241218: Installer incorrectly creates vocabulary. Both are now fixed. Back to porting.

fgm’s picture

Status: Needs work » Fixed

Changes in today's version:

  1. job_queue reference removed from classified_notifications: module is not (yet ?) ported to D7 so there is no "standard" equivalent Queue API consumer: emailing is synchronous, not queued, for now
  2. 3 tokens now available: [user:classified-ads-url] (chainable), [user:classified-ads] (themed) and [user:classified-ads-plain] (plaintext, for mail)
  3. token hooks in separate file
  4. modr8 reference removed: module does not (and will not) exist on Drupal 7 : use a workflow solution instead when needed
  5. Classified Link field formatter now works with any term, linking either to Classified Ads pages when they exist or standard taxonomy term pages otherwise
  6. Ad edit form now works, including jQuery counter
  7. installer no longer needs a manual field settings save and flush cache
  8. fixed 2 bugs found by @LagPRO

And with this, the actual porting is finished: rolling alpha1. Let the bug hunt commence. For any issues with this first alpha please open separate tickets, one per issue, to ease tracking.

Status: Fixed » Closed (fixed)

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