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

Weird AssertionError #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Weird AssertionError #9

wants to merge 1 commit into from

Conversation

pawl
Copy link

@pawl pawl commented Oct 5, 2013

This line was causing some AssertionErrors that didn't seem to make any sense. Is that line necessary? It seems to work fine when it's commented out to bypass the errors.

Traceback (most recent call last):
File "remoteInt3gSync.py", line 178, in
sp_list.save()
File "/usr/local/lib/python2.7/dist-packages/sharepoint/lists/init.py", line 322, in save
assert not any(row._changed for row in self.rows)
AssertionError

This line was causing some AssertionErrors that didn't seem to make any sense. Is that line necessary? It seems to work fine when it's commented out to bypass the errors.

Traceback (most recent call last):
  File "remoteInt3gSync.py", line 178, in <module>
    sp_list.save()
  File "/usr/local/lib/python2.7/dist-packages/sharepoint/lists/__init__.py", line 322, in save
    assert not any(row._changed for row in self.rows)
AssertionError
@pawl
Copy link
Author

pawl commented Oct 5, 2013

Ah, thanks for taking the time to write the process. That helped to find the problem.

I'm pretty sure this has something to do with the amount of data I'm saving. I'm trying to save around 500 rows. So, I added a limit of only saving 50 rows at a time and I no longer get the error message.

I'll do more debugging and try to find a fix.

@pawl pawl closed this May 22, 2014
@pawl pawl reopened this May 22, 2014
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

Successfully merging this pull request may close these issues.

2 participants