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

Delete Sentence #46

Closed
tiagocampo opened this issue May 7, 2014 · 1 comment
Closed

Delete Sentence #46

tiagocampo opened this issue May 7, 2014 · 1 comment
Assignees
Milestone

Comments

@tiagocampo
Copy link
Collaborator

Please, explain more

    # IT HAS A PROBLEM, IF MORE THEN ONE COLUMn WAS DISPLAYED.
    # But for now its ok. I dont know how to provide just some items.

This is not an issue, this is expected. We can easily correct by creating a deleteSentence function that receives all the sentence information.
# IT HAS ANOTHER PROBLEM: IT DELETES ALL SENTENCES THAT ARE EQUAL
# But, because the user has selected a section, sub section and function
# It was expected that just that sentence that he select.

@tiagocampo tiagocampo self-assigned this May 7, 2014
@tiagocampo tiagocampo added this to the Version_1.0.1 milestone May 7, 2014
@dpizetta
Copy link
Collaborator

dpizetta commented May 8, 2014

But the problem of the first one is that: if the column numbers varies the quantity of information varies too. Like If the table has displayed section and sentence, when you double clicked a sentence it returns section and sentence, but the method not treats this now.

The second if you read from selected categories, its true, but if you select all (not select anything) and select a sentence you dont have the information about categories.

tiagocampo added a commit that referenced this issue May 8, 2014
Fix issues #48 and #46.
Added a bunch of new functions to deal with sentences update and remove.
Now the listSentences function must return the entry's ID.
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