I ran the Coder (code review) module and it guided me through a few changes for porting to 6.

There are a few remaining issues:

      Line 458: new hook_theme() function is required to register theme_ functions (Drupal Docs)
      function theme_copyright_admin_overview($form) {

      severity: critical Line 85: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
        if ($may_cache) {

      severity: critical Line 86: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
          $items[] = array(

      severity: critical Line 93: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
          $items[] = array(

      severity: critical Line 99: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
          $items[] = array(

      severity: critical Line 106: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
          $items[] = array(

      severity: critical Line 112: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
         $items[] = array(

      severity: critical Line 533: hook_form_alter() parameters have changed (Drupal Docs)
      function copyright_form_alter($form_id, &$form) {

copyright.install

      severity: critical click to read more Line 17: A new schema API has been added in 6.x (Drupal Docs)
        switch ($GLOBALS['db_type']) {

      Explanation: This patch caused changes to the format of hook_install(), hook_uninstall(), and hook_update_N(). No longer are switch statements done on $GLOBALS['db_type']; instead, use the variety of schema API functions to perform table manipulation.

      severity: critical Line 176: db_next_id is deprecated. Use db_last_insert_id instead. (Drupal Docs)
            $cpyid = db_next_id("{copyrights}_cpyid");

      severity: critical click to read more Line 208: A new schema API has been added in 6.x (Drupal Docs)
        switch ($GLOBALS['db_type']) {

      Explanation: This patch caused changes to the format of hook_install(), hook_uninstall(), and hook_update_N(). No longer are switch statements done on $GLOBALS['db_type']; instead, use the variety of schema API functions to perform table manipulation.

      severity: critical click to read more Line 296: A new schema API has been added in 6.x (Drupal Docs)
        switch ($GLOBALS['db_type']) {
CommentFileSizeAuthor
copyright.diff4.82 KBreikiman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Robrecht Jacques’s picture

Version: 5.x-1.6 » master
Robrecht Jacques’s picture

Status: Active » Needs work
davebv’s picture

How is the deveolpement of this port? I am really interested. Thank you!

davebv’s picture

is there anybody working on this? thanks!

toemaz’s picture

In order to get this issue alive and kicking again, Webchick released a nice screencast how one can easily help porting this module: http://www.lullabot.com/node/397/play

Give it a try if you really want this module for D6.

Robrecht Jacques’s picture

Assigned: Unassigned » Robrecht Jacques

I've started the port. ETA Sunday August 10th.

davebv’s picture

thanks for your work!!

davebv’s picture

any news on this?

davebv’s picture

is the "diff" a patch for make it to work with drupal 6?

toemaz’s picture

@robrecht in case you have some code already for the D6 version of copyright, could you check it in? I have the time now to jump on this port job.

Robrecht Jacques’s picture

@toemaz: no I don't have any code yet. If you can update the patch, I'll commit and work together on solving the remaining issues. Thanks!

nirvanajyothi’s picture

will be useful. subscribing.

MTecknology’s picture

This port is important to the Ubuntu/Canonical community. I will be able to start work on it starting next week if it is still needed. However, if you think you can finish this in a timely fashion - I can let you work on nit.

toemaz’s picture

@MTecknology

I'm a user of this module as well on D5 and while considering to port it to 6, I was wondering whether this module has not become redundant if you consider that a CCK field can be used to add a license to a node as well. By overriding the theme for the license field, one can easily mimic the same behaviour of the copyright module. What's more: by using a CCK field, it will exposed to Views, Faceted Search, ... and it will be easier to maintain and upgrade the data when moving up to D7.

davebv’s picture

@toemaz
I think use this with integration with CCK could be a more than a great solution, maybe we can start a feature request.

MTecknology’s picture

Status: Needs work » Closed (fixed)

Feel free to reopen if this is still an issue. I'm just closing it because it's been inactive for a long time.