Skip to content

PodAPI :: ping_item (or similar name) #61

Open
sc0ttkclark opened this Issue Sep 15, 2011 · 10 comments

2 participants

@sc0ttkclark
Pods Foundation, Inc member

The idea is that this function will either +1 or -1 (actually, can be any number) a specific column of a specific item, instead of a full-blown save_pod_item request dealing with multiple queries and functionality.

Cuts right to the chase, one single query to update the table should do it.

@sc0ttkclark sc0ttkclark was assigned Sep 15, 2011
@mikevanwinkle

I'm on it.

@sc0ttkclark
Pods Foundation, Inc member

Awesome, but why'd you close it? Leave it open and let me close so I can commit to the 2.0 branch, I've gotta refresh 2.0 with 1.12 code and after that I can get any new stuff in there too. Maybe provide a gist of your solution?

https://gist.github.com/

@sc0ttkclark sc0ttkclark reopened this Sep 15, 2011
@mikevanwinkle

Damn, because it's too early (west coast) and I clicked the wrong button I guess. sorry bout that.

@sc0ttkclark
Pods Foundation, Inc member

Np, just a good time to figure out the process until 2.0 is workable again.

@sc0ttkclark
Pods Foundation, Inc member

HAHA! I clicked it now, it's early for me still..

@sc0ttkclark sc0ttkclark reopened this Sep 15, 2011
@sc0ttkclark
Pods Foundation, Inc member

I clicked Comment & Close

@sc0ttkclark
Pods Foundation, Inc member

I just removed the button with my AdBlock extension for Chrome, it targets it by the id/class and tag name. Now I'll never have that issue again!

@mikevanwinkle

I don't care what you call it, but at one time we had a function called pod_exists() didn't we ... which should have been Pod_API::pod_item_exists() ;-) ... anyway, couldn't we just add on a parameter to that function and change the name and create an option to return the value (instead of boolean) ?

Seems pretty simple to me.

@mikevanwinkle

They should really make that button smaller.

@sc0ttkclark
Pods Foundation, Inc member

Pod_API::pod_exists checks to see if an actual Pod exists, but I don't recall a pod_item_exists(), what sort of use do you think that would give?

@sc0ttkclark sc0ttkclark removed their assignment Dec 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.