D7 acquia_connector should also have this functionality

CommentFileSizeAuthor
#6 861232-D7-6.patch69 KBpwolanin
#3 acquia_connector-D7.patch73.07 KBdmitry_bezer
#1 acquia_connector-AN-17354-2.patch65.38 KBdmitry_bezer
acquia_connector-AN-17354-1.patch38.56 KBdmitry_bezer

Comments

dmitry_bezer’s picture

dmitry_bezer commented
StatusFileSize
new acquia_connector-AN-17354-2.patch65.38 KB

Current version of D6 connector ported to D7

ChrisRut’s picture

ChrisRut commented
Status: Needs review » Reviewed & tested by the community

Worked for me

dmitry_bezer’s picture

dmitry_bezer commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new acquia_connector-D7.patch73.07 KB

- Added a hook notifying other modules that subscription status has changed
- acquia_agent_streams.inc removed (content moved to acquia_agent.module)

pwolanin’s picture

pwolanin commented
Status: Needs review » Needs work

Looks pretty good, but we can get rid of acquia_agent_random_bytes() since we have such a function in Drupal 7 core.

pwolanin’s picture

pwolanin commented

Some other fixes needed - including a fatal error trying to include the deleted file.

pwolanin’s picture

pwolanin commented
Status: Needs work » Needs review
StatusFileSize
new 861232-D7-6.patch69 KB

Here's a mostly working patch with several fixes.

pwolanin’s picture

pwolanin commented
Status: Needs review » Fixed

committed that patch - lets make any more fixes in a follow-up issue.

Status: Fixed » Closed (fixed)

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