CVS edit link for hhaidoura

Prepare a new integration with RB CMS
CommentFileSizeAuthor
#6 pushconnector.tar_.gz6.54 KBhhaidoura
#1 pushconnector.tar_.gz1.09 KBhhaidoura

Comments

hhaidoura’s picture

hhaidoura commented
StatusFileSize
new pushconnector.tar_.gz1.09 KB
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Postponed (maintainer needs more info) » Needs review

Remember to change the status to needs review, otherwise reviewers will not know there is code to review.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Closed (won't fix) 
  1. function pushconnector_help($path, $arg) {
  switch ($path) {
    case 'admin/help#pushconnector':
      return '<p> '. t('Help!!') .' </p>';

    case 'admin/settings/connector':
      $output = '<p> '. t('RB connector description page.') .' </p>';      return $output;

  }
}

    The help is not useful; you should put the code after the semicolon to a new line.

  2. See the Drupal coding standards to understand how a module code should be written. 
  3.   version = 6.x-0.1

    That line should be removed from the .info file.

  4. The module is not complete, and it doesn't do anything. The settings page has not even been implemented.
hhaidoura’s picture

hhaidoura commented
Status: Closed (won't fix) » Needs review

The following module is a starting steps to communicate with the CMS remotely using drupal xmlrpc callbacks,
usually we used to pull data from the CMS and it was very painful due to speed complexities and other issues.
So the module will be help the CMS to push content into drupal an creating the nodes with its Binaries remotely without accessing drupal.

More implementation is done in the attachment below..
ofcourse still needs more work.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Closed (won't fix)

The CVS application has been already declined; re-opening this report doesn't change the status of the application. Feel free to re-apply for a CVS account when the code is complete.

hhaidoura’s picture

hhaidoura commented
Status: Closed (won't fix) » Needs review
StatusFileSize
new pushconnector.tar_.gz6.54 KB

more development.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Closed (won't fix)

See comment #5.