Any plans to port to Drupal 6.x?

CommentFileSizeAuthor
#12 weblinks_drupal_6.tar.gz17.88 KBrmiddle

Comments

bdragon’s picture

Assigned: Unassigned » bdragon

I'll be taking over maintainership, and part of that will be a Drupal 6 port.

joachim’s picture

Any news on this?

bird4ever’s picture

When will the module for drupal 6.x be ready?

rmiddle’s picture

Hopefully there will be a patch soon. :)

rmiddle’s picture

bdragon,

Are you looking for a Co-Maintainer?

Thanks
Robert

encho’s picture

subscribing

bdragon’s picture

OK, I totally dropped the ball here.
rmiddle: You'd have to get in touch with ideaoforder, I don't have access to twiddle with the cvs access list.

bdragon’s picture

Assigned: bdragon » Unassigned
underpressure’s picture

how bout a status update, so those o us waiting can have some idea of what is going on.

rmiddle’s picture

I am testing a patch right now assuming it goes well I will post it in the next few days. Then it is up to bdragon.

Thanks
Robert

rmiddle’s picture

Ok here is a tar.gz of the module for 6. I will work up a full patch later on.

Thanks
Robert

rmiddle’s picture

StatusFileSize
new17.88 KB

Lets try this again.

rmiddle’s picture

Assigned: bdragon » Unassigned
Status: Needs review » Active

Stupid typo. Don't use number 12.

rmiddle’s picture

Ok here is the D6 patch.

bdragon’s picture

Assigned: Unassigned » bdragon
Status: Active » Needs review
Douglas’s picture

Is there any chance you could supply the patched files, cause I haven't a clue what to do with the patch file?

Douglas

nancydru’s picture

@douglas: isn't that what #13 is?

nancydru’s picture

@rmiddle: The "Maximum Recent Web Links" setting displays after the buttons.

On creating a new link, the weight defaults to -10; it should be zero. You have "#maxlength" specified on the two "weight" elements; it should be "#delta". Plus you should have '#default_value' => isset($node->weight) ? $node->weight : 0,

Breadcrumbs shows: Array › Array › Array

  // Breadcrumb navigation
  $breadcrumb = array(l(t('Home'), null));
  $breadcrumb[] = l($vocabulary->name, 'weblinks');
  if ($parents = taxonomy_get_parents_all($node->tid)) {
    foreach ($parents as $p) {
      $breadcrumb[] = l($p->name, 'weblinks/'. $p->tid);
    }
  }
  drupal_set_breadcrumb($breadcrumb);

    Exists in two places.

The install file should have the updates removed. They are for 5.x and no longer applicable - and numbered incorrectly for 6.x.

Douglas’s picture

Isn't 14 a patch of 13?

nancydru’s picture

#13 is the whole module, pretty much as "download" on the project page would be.

rmiddle’s picture

Douglas,

This is patch is in CVS. Not sure why BDragon hasn't done a dev release yet but it is in there with a few fixes. It is working fine on my site.

rmiddle’s picture

Just talked with BDragon and he has added it to his todo so it should be released sometime this week.

Thanks
Robert

rmiddle’s picture

Assigned: Unassigned » bdragon
Status: Active » Needs review

I removed the attachments as these are in CVS already and have a few mistakes that are fixed in CVS.

Thanks
Robert

metabits’s picture

subscribing...

and waiting for it!

thanks all of you in advance

rmiddle’s picture

Status: Needs review » Fixed

Aplha1 for 6 has been released. Closing this ticket. Please submit issue Outside of the one I already know about.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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