Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

has_content? function is not reliable with loops #1

Open
mjpete3 opened this issue Dec 5, 2012 · 2 comments
Open

has_content? function is not reliable with loops #1

mjpete3 opened this issue Dec 5, 2012 · 2 comments

Comments

@mjpete3
Copy link
Owner

mjpete3 commented Dec 5, 2012

The has_content? function does not return a reliably when dealing with loops. Recursive structure causes the function to run through the entire structure and not short circuit itself.

@mjpete3
Copy link
Owner Author

mjpete3 commented Mar 22, 2013

Sean,

I actually dug into the code. figured out what I was doing wrong.
Also I've been maintaining a separate gem: pd_x12. I've had several
people ask for additional transactions.

If you are ok, I would like to take a look at what you did with the
loops and merge it into the gem I'm maintaining.

Marty

On Wed, 2013-03-20 at 05:30 -0700, Sean Walberg wrote:

Hi,

I'm sorry, I didn't see this issue before. Do you have a small sample
to duplicate the problem? I recently merged some changes that deal
with loops, too.


Reply to this email directly or view it on GitHub.

@swalberg
Copy link

Hi Marty,

For sure!

Alternatively, did you want to submit them as pull requests? I've been
talking with the original author of the gem and have moved my repo to an
organization (x12ruby/x12) and will soon be getting access to update the
x12 gem over on RubyGems.org. He's basically abandoned the project.

Thanks,

Sean

On Fri, Mar 22, 2013 at 8:53 AM, MJPete3 notifications@github.com wrote:

Sean,

I actually dug into the code. figured out what I was doing wrong.
Also I've been maintaining a separate gem: pd_x12. I've had several
people ask for additional transactions.

If you are ok, I would like to take a look at what you did with the
loops and merge it into the gem I'm maintaining.

Marty

On Wed, 2013-03-20 at 05:30 -0700, Sean Walberg wrote:

Hi,

I'm sorry, I didn't see this issue before. Do you have a small sample
to duplicate the problem? I recently merged some changes that deal
with loops, too.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-15297599
.

Sean Walberg sean@ertw.com http://ertw.com/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants